Forge version 1.2.16
Post MTG Forge Related Programming Questions Here
Moderators: timmermac, Agetian, friarsol, Blacksmith, KrazyTheFox, CCGHQ Admins
70 posts
• Page 3 of 5 • 1, 2, 3, 4, 5
Re: Forge version 1.2.16
by Doublestrike » 16 Oct 2012, 00:19
Yep, now that I don't have to focus on "getting things working" I can focus on "getting things looking good." Should be fun for everyone involved.Chris H. wrote:Appears to be more buttony in appearance.
it looks good on my Mac for the most part.
Whew. Great news!
That was supposed to be a drop shadow effect to add some depth to the title...if people don't like it I'll try something else, let me know everyone.The title label for the views look off.
Your screenshot does show something else though - the selected menu (constructed) is supposed to have the same color/texture as the display area. Apparently it's not painting properly. Good to know. Screenshots are always helpful!
Ahh ok will get that working very soon.The Exit Forge button did not work...
---
A joke is a very serious thing.
A joke is a very serious thing.
-

Doublestrike - UI Programmer
- Posts: 715
- Joined: 08 Aug 2011, 09:07
- Location: Bali
- Has thanked: 183 times
- Been thanked: 161 times
Re: Forge version 1.2.16
by friarsol » 16 Oct 2012, 00:59
I basically can't start a game (of any type) at all on my laptop because the decklists (or other things on the top of the screen) are much larger than they used to be, and the start button is pushed off screen.
Was there more preferred files I was s'posed to delete? I really hope this is going to be resolved.
http://imgur.com/2HdBM
http://imgur.com/5J9bV
Was there more preferred files I was s'posed to delete? I really hope this is going to be resolved.
http://imgur.com/2HdBM
http://imgur.com/5J9bV
- friarsol
- Global Moderator
- Posts: 7593
- Joined: 15 May 2010, 04:20
- Has thanked: 243 times
- Been thanked: 965 times
Re: Forge version 1.2.16
by Doublestrike » 16 Oct 2012, 01:22
@Chris - exit button fixed.
@Friarsol - Ahh ok yes sorry I know exactly the problem. Give me a half hour and I think I can nail it.
(just FYI only the match UI has the "preferred" issue. Only one file needs/needed to be reset, match_preferred.xml.)
EDIT
@sol - Should be functional now, but not yet beautiful.
@Friarsol - Ahh ok yes sorry I know exactly the problem. Give me a half hour and I think I can nail it.
(just FYI only the match UI has the "preferred" issue. Only one file needs/needed to be reset, match_preferred.xml.)
EDIT
@sol - Should be functional now, but not yet beautiful.
---
A joke is a very serious thing.
A joke is a very serious thing.
-

Doublestrike - UI Programmer
- Posts: 715
- Joined: 08 Aug 2011, 09:07
- Location: Bali
- Has thanked: 183 times
- Been thanked: 161 times
Re: Forge version 1.2.16
by Chris H. » 16 Oct 2012, 01:37
Doublestrike wrote:That was supposed to be a drop shadow effect to add some depth to the title...if people don't like it I'll try something else, let me know everyone.Chris H. wrote:The title label for the views look off.
If possible, can you take another look at my screenshot.
The title label is being pushed to the right by about 100 pixels or so. My screenshot is full sized and notice how the right side of the title label is being drawn off screen. I did not cut off that portion of my screen while doing the screenshot.
I suspect that if a vertical scrollbar is being drawn it is being drawn off screen.
The title text itself is not centered inside of the title label but that is not as much of a problem.
-

Chris H. - Forge Moderator
- Posts: 6320
- Joined: 04 Nov 2008, 12:11
- Location: Mac OS X Yosemite
- Has thanked: 644 times
- Been thanked: 643 times
Re: Forge version 1.2.16
by Doublestrike » 16 Oct 2012, 01:48
@Chris - all of that is on purpose
Maybe it's not interpreted as I intended...do other people have the same "off-kilter" feeling Chris is describing?
Maybe it's not interpreted as I intended...do other people have the same "off-kilter" feeling Chris is describing?
---
A joke is a very serious thing.
A joke is a very serious thing.
-

Doublestrike - UI Programmer
- Posts: 715
- Joined: 08 Aug 2011, 09:07
- Location: Bali
- Has thanked: 183 times
- Been thanked: 161 times
Re: Forge version 1.2.16
by Max mtg » 16 Oct 2012, 21:52
Ok, I have removed AllZone.getXXXPlayer, but the game won't start now 
won't commit that until its playable.
The main problem is: on launch UI tries to subscribe to events from MagicStack and other classes that do not exist until the game is started.
won't commit that until its playable.
The main problem is: on launch UI tries to subscribe to events from MagicStack and other classes that do not exist until the game is started.
Single class for single responsibility.
- Max mtg
- Programmer
- Posts: 1997
- Joined: 02 Jul 2011, 14:26
- Has thanked: 173 times
- Been thanked: 334 times
Re: Forge version 1.2.16
by Doublestrike » 17 Oct 2012, 00:30
Draft submenu has been updated.
Besides the aesthetics, a few architectural changes:
- No more list of names
- Start button area has radio button for "single" or "gauntlet" choice
- "Directions" panel removed
- Concise "Directions" blurb added
Sealed submenu has also been updated.
Besides the aesthetics, a few architectural changes:
- No more list of names
- Start button area has radio button for "single" or "gauntlet" choice
- "Directions" panel removed
- Concise "Directions" blurb added
Sealed submenu has also been updated.
---
A joke is a very serious thing.
A joke is a very serious thing.
-

Doublestrike - UI Programmer
- Posts: 715
- Joined: 08 Aug 2011, 09:07
- Location: Bali
- Has thanked: 183 times
- Been thanked: 161 times
Re: Forge version 1.2.16
by Doublestrike » 17 Oct 2012, 01:07
I can have a look at this if you want; what class is causing trouble?Max mtg wrote:The main problem is: on launch UI tries to subscribe to events from MagicStack and other classes that do not exist until the game is started.
---
A joke is a very serious thing.
A joke is a very serious thing.
-

Doublestrike - UI Programmer
- Posts: 715
- Joined: 08 Aug 2011, 09:07
- Location: Bali
- Has thanked: 183 times
- Been thanked: 161 times
Re: Forge version 1.2.16
by Doublestrike » 17 Oct 2012, 07:36
Quest Duels and Challenges menu has been partially updated.
I had to make a few changes to their control classes, please alert if problems.
I had to make a few changes to their control classes, please alert if problems.
---
A joke is a very serious thing.
A joke is a very serious thing.
-

Doublestrike - UI Programmer
- Posts: 715
- Joined: 08 Aug 2011, 09:07
- Location: Bali
- Has thanked: 183 times
- Been thanked: 161 times
Re: Forge version 1.2.16
by Max mtg » 17 Oct 2012, 09:18
Sure.Doublestrike wrote:I can have a look at this if you want; what class is causing trouble?Max mtg wrote:The main problem is: on launch UI tries to subscribe to events from MagicStack and other classes that do not exist until the game is started.
I'll commit a branch (if my account has enough priveledges) in 8-9 hours.
Although that would mean developing a branch, there's already no way to avoid it.
Single class for single responsibility.
- Max mtg
- Programmer
- Posts: 1997
- Joined: 02 Jul 2011, 14:26
- Has thanked: 173 times
- Been thanked: 334 times
Re: Forge version 1.2.16
by friarsol » 17 Oct 2012, 12:13
If you are so averse to creating a branch why not just create a patch of your code and post it up on the forums?Max mtg wrote:Sure.Doublestrike wrote:I can have a look at this if you want; what class is causing trouble?Max mtg wrote:The main problem is: on launch UI tries to subscribe to events from MagicStack and other classes that do not exist until the game is started.
I'll commit a branch (if my account has enough priveledges) in 8-9 hours.
Although that would mean developing a branch, there's already no way to avoid it.
- friarsol
- Global Moderator
- Posts: 7593
- Joined: 15 May 2010, 04:20
- Has thanked: 243 times
- Been thanked: 965 times
Re: Forge version 1.2.16
by Max mtg » 17 Oct 2012, 20:38
Branch is not a big problem, yet will have to move changesets from trunk. But now making a branch seems to be inevitable.
I thought my project would not affect code that deep.
To see my efforts, switch to /branches/hotseat/
Now the game crashes on end of round, since game results collection is not working yet.
I thought my project would not affect code that deep.
To see my efforts, switch to /branches/hotseat/
Now the game crashes on end of round, since game results collection is not working yet.
Single class for single responsibility.
- Max mtg
- Programmer
- Posts: 1997
- Joined: 02 Jul 2011, 14:26
- Has thanked: 173 times
- Been thanked: 334 times
Re: Forge version 1.2.16
by Doublestrike » 18 Oct 2012, 00:06
So, the issue you mentioned before is OK now?Max mtg wrote:Now the game crashes on end of round, since game results collection is not working yet.
---
A joke is a very serious thing.
A joke is a very serious thing.
-

Doublestrike - UI Programmer
- Posts: 715
- Joined: 08 Aug 2011, 09:07
- Location: Bali
- Has thanked: 183 times
- Been thanked: 161 times
Re: Forge version 1.2.16
by Max mtg » 18 Oct 2012, 05:38
Yes, as I moved subscription to game-related events from constructor to a dedicated method for CMessage, CLog and CStack and called that method at the beginning of a match, the problem mentioned yesterday has gone.Doublestrike wrote:So, the issue you mentioned before is OK now?Max mtg wrote:Now the game crashes on end of round, since game results collection is not working yet.
Single class for single responsibility.
- Max mtg
- Programmer
- Posts: 1997
- Joined: 02 Jul 2011, 14:26
- Has thanked: 173 times
- Been thanked: 334 times
Re: Forge version 1.2.16
by Max mtg » 18 Oct 2012, 06:44
So, is the trunk version playable now?
Single class for single responsibility.
- Max mtg
- Programmer
- Posts: 1997
- Joined: 02 Jul 2011, 14:26
- Has thanked: 173 times
- Been thanked: 334 times
70 posts
• Page 3 of 5 • 1, 2, 3, 4, 5
Who is online
Users browsing this forum: No registered users and 10 guests