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

change text on viewport window #6553

Closed
DorpsGek opened this issue Apr 3, 2017 · 5 comments
Closed

change text on viewport window #6553

DorpsGek opened this issue Apr 3, 2017 · 5 comments
Labels
bug Something isn't working flyspray This issue is imported from FlySpray (https://bugs.openttd.org/)

Comments

@DorpsGek
Copy link
Member

DorpsGek commented Apr 3, 2017

hackalittlebit opened the ticket and wrote:

Maybe it is only me but it always confuses me this
'Copy to viewport' and 'Paste from viewport'
I tried to give a more descriptive text.
'Copy to viewport' was changed to 'Scroll to main' and
'Paste from viewport' to 'Scroll main window'
I do not think it is necessary to mention the viewport because the buttons are on the viewport.
See patch below.

Attachments

Reported version: Version?
Operating system: All


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

DorpsGek commented Apr 4, 2017

adf88 wrote:

I must admit that it was always confusing to me too. But the proposed changes are not much better IMO. Perhaps these buttons should be graphical? Quick mock-up:

Attachments


This comment was imported from FlySpray: https://bugs.openttd.org/task/6553#comment14417

@DorpsGek
Copy link
Member Author

DorpsGek commented Apr 4, 2017

hackalittlebit wrote:

Maybe this is a better try.
See patch
Graphics are so small on my 4000k screen.:-(

Attachments


This comment was imported from FlySpray: https://bugs.openttd.org/task/6553#comment14418

@DorpsGek
Copy link
Member Author

DorpsGek commented Apr 5, 2017

peter1138 wrote:

It's still ambiguous


This comment was imported from FlySpray: https://bugs.openttd.org/task/6553#comment14421

@DorpsGek
Copy link
Member Author

DorpsGek commented Apr 5, 2017

hackalittlebit wrote:

Agreed.
This however should be better.
Note buttons were swapped.

Attachments


This comment was imported from FlySpray: https://bugs.openttd.org/task/6553#comment14422

@DorpsGek DorpsGek added Core flyspray This issue is imported from FlySpray (https://bugs.openttd.org/) labels Apr 7, 2018
@andythenorth andythenorth added bug Something isn't working and removed enhancement from FlySpray labels Apr 13, 2018
@andythenorth
Copy link
Contributor

I am labelling this one a bug. I thought 'copy to viewport' and 'paste from viewport' actually copied the tile contents (I was confused how anyone had managed to achieve that or why it was useful).

@frosch123 frosch123 removed the Core label Apr 14, 2018
Alberth289346 added a commit to Alberth289346/OpenTTD that referenced this issue Jun 24, 2018
Alberth289346 added a commit to Alberth289346/OpenTTD that referenced this issue Jun 24, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working flyspray This issue is imported from FlySpray (https://bugs.openttd.org/)
Projects
None yet
Development

No branches or pull requests

3 participants