Page 1 of 1

BUG: No replay button for the first level

PostPosted: 02 Apr 2012, 18:27
by Tedium
The replay button (background and icon) of the first level of a levelpack doesn't appear when you select it :(.
The actual button still works though.

Replaybug.png
Replaybug.png (54.95 KiB) Viewed 4474 times

As you can see I'm trying out new "inky" icons inspired by odamite's translation mock-ups. See the attachment. Feedback is welcome.

Re: BUG: No replay button for the first level

PostPosted: 02 Apr 2012, 18:32
by ctdabomb
what version?

Re: BUG: No replay button for the first level

PostPosted: 02 Apr 2012, 18:40
by Tedium
0.3-2.4 from the openSUSE repositories.

Re: BUG: No replay button for the first level

PostPosted: 02 Apr 2012, 20:07
by odamite
Hello Tedium,

I see the bug and I'm not sure what is causing it. And I'm not quite sure what is version 0.3-2.4... :( I'm trying to see what's causing this and I'm sure Edward_Lii can help with it.

I love the new icons and I'm happy that I've been the inspiration for you. It's quite funny because I created the mockups very fast and I was too lazy to find the real icons there. :D But good work and lets see will we start using them.

EDIT: I know what is causing it:
Line 287 (might not be exactly) in LevelPlaySelect.cpp
{l Code}: {l Select All Code}
      if(selectedNumber->getNumber()>0 && selectedNumber->getNumber()<levels.getLevelCount()) {

to
{l Code}: {l Select All Code}
      if(selectedNumber->getNumber()>=0 && selectedNumber->getNumber()<levels.getLevelCount()) {

1 letter changed. Fixed, but I'm not sure can I commit it now or not...

Re: BUG: No replay button for the first level

PostPosted: 03 Apr 2012, 16:14
by Edward_Lii
Hello all,

odamite {l Wrote}:EDIT: I know what is causing it:
Line 287 (might not be exactly) in LevelPlaySelect.cpp
...
1 letter changed. Fixed, but I'm not sure can I commit it now or not...

It's fixed in the latest revision (321), thanks for not committing it. :D