It is currently 30 Oct 2025, 06:43
   
Text Size

Forge version 1.2.16

Post MTG Forge Related Programming Questions Here

Moderators: timmermac, Agetian, friarsol, Blacksmith, KrazyTheFox, CCGHQ Admins

Re: Forge version 1.2.16

Postby Doublestrike » 16 Oct 2012, 00:19

Chris H. wrote:Appears to be more buttony in appearance.
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.

it looks good on my Mac for the most part.


Whew. Great news!

The title label for the views look off.
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.

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! =D>

The Exit Forge button did not work...
Ahh ok will get that working very soon.
---
A joke is a very serious thing.
User avatar
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

Postby 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
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

Postby 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.
---
A joke is a very serious thing.
User avatar
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

Postby Chris H. » 16 Oct 2012, 01:37

Doublestrike wrote:
Chris H. wrote:The title label for the views look off.
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.
 
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. :)
User avatar
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

Postby Doublestrike » 16 Oct 2012, 01:48

@Chris - all of that is on purpose :lol:

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.
User avatar
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

Postby 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.
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

Postby 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.
---
A joke is a very serious thing.
User avatar
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

Postby Doublestrike » 17 Oct 2012, 01:07

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 can have a look at this if you want; what class is causing trouble?
---
A joke is a very serious thing.
User avatar
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

Postby 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.
---
A joke is a very serious thing.
User avatar
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

Postby Max mtg » 17 Oct 2012, 09:18

Doublestrike wrote:
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 can have a look at this if you want; what class is causing trouble?
Sure.
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

Postby friarsol » 17 Oct 2012, 12:13

Max mtg wrote:
Doublestrike wrote:
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 can have a look at this if you want; what class is causing trouble?
Sure.
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.
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?
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

Postby 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.
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

Postby Doublestrike » 18 Oct 2012, 00:06

Max mtg wrote:Now the game crashes on end of round, since game results collection is not working yet.
So, the issue you mentioned before is OK now?
---
A joke is a very serious thing.
User avatar
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

Postby Max mtg » 18 Oct 2012, 05:38

Doublestrike wrote:
Max mtg wrote:Now the game crashes on end of round, since game results collection is not working yet.
So, the issue you mentioned before is OK now?
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.
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

Postby 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

PreviousNext

Return to Developer's Corner

Who is online

Users browsing this forum: No registered users and 6 guests

Main Menu

User Menu

Our Partners


Who is online

In total there are 6 users online :: 0 registered, 0 hidden and 6 guests (based on users active over the past 10 minutes)
Most users ever online was 9298 on 10 Oct 2025, 12:54

Users browsing this forum: No registered users and 6 guests

Login Form