Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Move sprite ten units at a time in sprite aligner if ctrl is pressed #6241

Closed
DorpsGek opened this issue Mar 1, 2015 · 2 comments
Closed

Move sprite ten units at a time in sprite aligner if ctrl is pressed #6241

DorpsGek opened this issue Mar 1, 2015 · 2 comments
Labels
component: interface This is an interface issue flyspray This issue is imported from FlySpray (https://bugs.openttd.org/) patch from FlySpray This issue is in fact a Patch, but imported from FlySrpay

Comments

@DorpsGek
Copy link
Member

DorpsGek commented Mar 1, 2015

juzza1 opened the ticket and wrote:

It would be useful to move the sprite in larger chuncks, especially when starting the alignment process from scratch. This patch adds extra functionality to the movement buttons in sprite aligner - if ctrl is pressed simultaneously when clicking, the offset is changed ten units at a time.

Attachments

Reported version: trunk
Operating system: All

This issue was imported from FlySpray: https://bugs.openttd.org/task/6241
@DorpsGek
Copy link
Member Author

DorpsGek commented Mar 4, 2015

planetmaker wrote:

Some random, unfiltered thoughts:

planetmaker: how does #6241 sound to you? I would expect 12 to be a better number than 10
Alberth, it's on my list of patches which sound very useful to me. Maybe 12 is better indeed
or 16 :)
or 8
hm...
8 ?
64 / 8 = 8; 32 / 8 = 4
having more buttons could be useful too, there is plenty of space
except printing the count at a sane position may be troublesome
<< and >> additionally to < and > ?
good idea
however, all that is additional to what juzza1 wrote, I think
these thoughts should not hinder us to commit his patch
i wondered whether it should consider the current zoom level of the main viewport or something
it makes no sense to move sprites by fractional pixels of the current zoom level
seems fair
but i am not sure whether aligning is always done via the main viewport :p
anyway, 16 makes more sense than 10, due to the zoom levels
should I just paste this IRC convo to that task? ;)
i.e. 16 at zi4 = 4 at zi1
yup, 16 or 8
fine by me planetmaker

This comment was imported from FlySpray: https://bugs.openttd.org/task/6241#comment13804

@DorpsGek
Copy link
Member Author

Alberth closed the ticket.

Reason for closing: Implemented

in r27451

This comment was imported from FlySpray: https://bugs.openttd.org/task/6241

@DorpsGek DorpsGek added component: interface This is an interface issue flyspray This issue is imported from FlySpray (https://bugs.openttd.org/) patch from FlySpray This issue is in fact a Patch, but imported from FlySrpay labels Apr 7, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
component: interface This is an interface issue flyspray This issue is imported from FlySpray (https://bugs.openttd.org/) patch from FlySpray This issue is in fact a Patch, but imported from FlySrpay
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@DorpsGek