Bug Reports (snapshot builds)
Post MTG Forge Related Programming Questions Here
Moderators: timmermac, Blacksmith, KrazyTheFox, Agetian, friarsol, CCGHQ Admins
Re: Bug Reports (snapshot builds)
by Chris H. » 28 Jan 2013, 22:56
myk wrote:did this happen recently? when did you notice it? What JVM are you running (I see a resolved bug that mentions this for the JDK)?Chris H. wrote:When adjusting the column widths the mouse pointer used to change when it was hovered over the titles. This feature has apparently dissapeared.
OS: Mac OS X Version: 10.7.5 Architecture: x86_64
Java Version: 1.6.0_37 Vendor: Apple Inc.
I am not sure when this situation developed. Reading the commit logs this morning convinced me to check out the changes in the deck editor.

We had a similar problem happen months ago and a number of people commented on not being able to change the widths of the columns. In reality they could but they had not tried. It was a case where the lack of the mouse pointer changing lead them to believe that this changing column widths feature had been removed or was broken.
-
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: Bug Reports (snapshot builds)
by myk » 28 Jan 2013, 23:57
I am actually seeing this same issue on both OSX 10.8 (same jvm version) and Linux (OpenJDK 1.11.5), but AFAIR, this has been been there for a while -- I thought the columns couldn't be resized before I tried the first time too.Chris H. wrote:OS: Mac OS X Version: 10.7.5 Architecture: x86_64
Java Version: 1.6.0_37 Vendor: Apple Inc.
I am not sure when this situation developed. Reading the commit logs this morning convinced me to check out the changes in the deck editor.
We had a similar problem happen months ago and a number of people commented on not being able to change the widths of the columns. In reality they could but they had not tried. It was a case where the lack of the mouse pointer changing lead them to believe that this changing column widths feature had been removed or was broken.
- myk
- Posts: 439
- Joined: 17 Jan 2013, 02:39
- Location: California
- Has thanked: 38 times
- Been thanked: 57 times
Re: Bug Reports (snapshot builds)
by Chris H. » 29 Jan 2013, 01:39
myk wrote:I am actually seeing this same issue on both OSX 10.8 (same jvm version) and Linux (OpenJDK 1.11.5), but AFAIR, this has been been there for a while -- I thought the columns couldn't be resized before I tried the first time too.
You are right.
The last beta that had the changeable mouse pointer is Forge Beta: 10-26-2012 ver 1.3.0 which is based on the code found in SVN rev 17721.
I would not lose too much sleep over it, if this missing feature can be added back in then great, but this is not a mission critical feature ... but it would be a good thing if we can add it back in.
If you are curious then checkout the earlier beta to see what it looked like.
-
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: Bug Reports (snapshot builds)
by myk » 29 Jan 2013, 01:49
while I'm waiting for feedback on the deck editor filters, I'll see if I can do some bisection and figure out which commit broke the functionalityChris H. wrote:The last beta that had the changeable mouse pointer is Forge Beta: 10-26-2012 ver 1.3.0 which is based on the code found in SVN rev 17721.
- myk
- Posts: 439
- Joined: 17 Jan 2013, 02:39
- Location: California
- Has thanked: 38 times
- Been thanked: 57 times
Re: Bug Reports (snapshot builds)
by myk » 29 Jan 2013, 02:08
hrm, I found the problem, but I need to investigate the implications of fixing it -- it's line 87 of FView.java:
Edit: TARGETING_LAYER is defined as JLayeredPane.MODAL_LAYER - 1, which seems to be playing games with internal constants. If I replace the above line with:
- Code: Select all
lpnDocument.add(TargetingOverlay.SINGLETON_INSTANCE.getPanel(), TARGETING_LAYER);
Edit: TARGETING_LAYER is defined as JLayeredPane.MODAL_LAYER - 1, which seems to be playing games with internal constants. If I replace the above line with:
- Code: Select all
lpnDocument.add(TargetingOverlay.SINGLETON_INSTANCE.getPanel(), JLayeredPane.MODAL_LAYER);
- myk
- Posts: 439
- Joined: 17 Jan 2013, 02:39
- Location: California
- Has thanked: 38 times
- Been thanked: 57 times
Re: Bug Reports (snapshot builds)
by RumbleBBU » 29 Jan 2013, 10:10
Asellia, apologies for not replying sooner. I have been busy fixing other things.Asellia wrote:I have had this trouble since 1.3.4 or so, I think, but it appears that I cannot partake in challenges. I have been using a series of custom worlds I made, utilizing default enemies/challenges, however, I cannot seem to pick challenges even when I have enough wins:
Edit:
I was using "forge-1.3.6-20130123.120651-5" as reference
Could you please zip and post your res/quest/data folder (from a situation where you ought to have got a challenge and aren't getting one)? I really want to solve the problem, and that could speed up the process considerably.
Re: Bug Reports (snapshot builds)
by myk » 29 Jan 2013, 19:16
A little more on this - changing the layer index is just part of the fix, we also have to resize it in FView.java. Otherwise the targeting arrows don't appear in the match (now that I looked more into how the targeting layer is used, I know what to look for). I'm testing it out in my branch and will get it merged when I merge the rest of the code there.myk wrote:hrm, I found the problem, but I need to investigate the implications of fixing it -- it's line 87 of FView.java:commenting this line out gets the resize cursor back, but I need to figure out while it's there in the first place : )
- Code: Select all
lpnDocument.add(TargetingOverlay.SINGLETON_INSTANCE.getPanel(), TARGETING_LAYER);
Edit: TARGETING_LAYER is defined as JLayeredPane.MODAL_LAYER - 1, which seems to be playing games with internal constants. If I replace the above line with:everything /seems/ to work -- the start game overlay looks ok, and targeting highlights look ok. On the other hand, I'm not entirely sure I know what to look for to decide whether those overlays are behaving as expected. Could someone else who knows a little more about this test the fix out?
- Code: Select all
lpnDocument.add(TargetingOverlay.SINGLETON_INSTANCE.getPanel(), JLayeredPane.MODAL_LAYER);
- myk
- Posts: 439
- Joined: 17 Jan 2013, 02:39
- Location: California
- Has thanked: 38 times
- Been thanked: 57 times
Re: Bug Reports (snapshot builds)
by Sloth » 30 Jan 2013, 19:09
Fixed! The generated trigger was just missing the player clause.friarsol wrote:r19213
Hands of Binding ciphered on my Mogg Flunkies. It doesn't have trample. I attack with it and an 8/8 Trample. AI blocks some on my trampler and 2/2 Wither to my Flunkie. Cipher on the Flunkie triggers (incorrectly, since I didn't damage a player).
-
Sloth - Programmer
- Posts: 3498
- Joined: 23 Jun 2009, 19:40
- Has thanked: 125 times
- Been thanked: 507 times
Re: Bug Reports (snapshot builds)
by Sloth » 30 Jan 2013, 19:35
Fixed! Thanks sol.friarsol wrote:Is Draft or Sealed working for anyone else?
-
Sloth - Programmer
- Posts: 3498
- Joined: 23 Jun 2009, 19:40
- Has thanked: 125 times
- Been thanked: 507 times
Re: Bug Reports (snapshot builds)
by myk » 30 Jan 2013, 22:38
I just checked in a number of changes to the deck editor in trunk. please report any bugs you run into here. I also found that the font for the default skin lacked some internal metadata that was causing text on all labels to be permanently top-aligned instead of centered. There were a lot of fonts that work fine, but I tried to choose one that looked as similar as possible to the font that was there. Please tell me if the new font causes any trouble for anyone!
- myk
- Posts: 439
- Joined: 17 Jan 2013, 02:39
- Location: California
- Has thanked: 38 times
- Been thanked: 57 times
Re: Bug Reports (snapshot builds)
by myk » 30 Jan 2013, 23:01
hey Sloth -- I reverted r19291 for now -- the change to EditorTableView. It was preventing the catalog from updating in the quest deck editor and spell shop.
Edit: I found the root cause -- it was the change I made last week that reapplies the filters when the deck editor is reloaded. I'll find a better place to put that line.
Edit 2: I found a better place to reapply the filters that works for all modes. Sorry for the inconvenience!
Edit: I found the root cause -- it was the change I made last week that reapplies the filters when the deck editor is reloaded. I'll find a better place to put that line.
Edit 2: I found a better place to reapply the filters that works for all modes. Sorry for the inconvenience!
- myk
- Posts: 439
- Joined: 17 Jan 2013, 02:39
- Location: California
- Has thanked: 38 times
- Been thanked: 57 times
Re: Bug Reports (snapshot builds)
by friarsol » 31 Jan 2013, 03:31
The ETB Ability of Overlaid Terrain has a "0:" before the "As ~ ETB" in the description. Looks like this is related to the ETB conversion
- friarsol
- Global Moderator
- Posts: 7593
- Joined: 15 May 2010, 04:20
- Has thanked: 243 times
- Been thanked: 965 times
Re: Bug Reports (snapshot builds)
by swordshine » 31 Jan 2013, 04:42
Fixed. Just forgot to remove the cost.friarsol wrote:The ETB Ability of Overlaid Terrain has a "0:" before the "As ~ ETB" in the description. Looks like this is related to the ETB conversion
- swordshine
- Posts: 682
- Joined: 11 Jul 2010, 02:37
- Has thanked: 116 times
- Been thanked: 87 times
Re: Bug Reports (snapshot builds)
by swordshine » 31 Jan 2013, 04:52
A subability Effect using param "RememberObjects$ Remembered" makes the effect remember redundant information, e.g. Circle of Protection.
Edit: Shortcut "E" is not functional.
Edit: Shortcut "E" is not functional.
- swordshine
- Posts: 682
- Joined: 11 Jul 2010, 02:37
- Has thanked: 116 times
- Been thanked: 87 times
Re: Bug Reports (snapshot builds)
by friarsol » 31 Jan 2013, 13:16
What does shortcut E mean?swordshine wrote:Edit: Shortcut "E" is not functional.
- friarsol
- Global Moderator
- Posts: 7593
- Joined: 15 May 2010, 04:20
- Has thanked: 243 times
- Been thanked: 965 times
Who is online
Users browsing this forum: No registered users and 20 guests