Skip to content

fixed #656 #740

Closed
wants to merge 1 commit into from

3 participants

@yofreke
Collaborator
yofreke commented Mar 4, 2012

try 2

@danielduner

Can you explain the changes in a few words? Just to make sure that your changes do what they are supposed to do :) Thank you :)

@yofreke
Collaborator
yofreke commented Mar 4, 2012

Always with the words... d:
This makes it so that the minimaps contained in a level button have a set size they display at (64 px). It then makes minimaps with a dimension greater than default turn into a largemap. A 64x64 portion of the largemap is displayed, offset by the buttons scrollX and scrollY. You can change the scroll values by hovering over corresponding edges of the displayed minimap.
Also fixed, is the poor look of a disabled page button while hovering. It adds a new "enabled" flag to clickable components, and not enabled buttons will have a different background rendered. This is implemented in the level select gui, where the page front and back buttons are.

@Elosanda
Elosanda commented Mar 4, 2012

are smaller maps also centered in the level select buttons? would be nice

@danielduner

It would be better to make separate pull requests for the minimap and for the buttons.

But both changes sound great.
+1 on merging, as soon as someone has tested it locally

@yofreke
The change can no longer be automatically merged. Can you merge with HEAD to update this? Thank you :)

@yofreke yofreke closed this Mar 5, 2012
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Something went wrong with that request. Please try again.