Skip to content

Removes redundant "Zoom In/Out" entries in the View menu#1491

Merged
cameronwhite merged 2 commits into
PintaProject:masterfrom
pedropaulosuzuki:master
May 27, 2025
Merged

Removes redundant "Zoom In/Out" entries in the View menu#1491
cameronwhite merged 2 commits into
PintaProject:masterfrom
pedropaulosuzuki:master

Conversation

@pedropaulosuzuki

Copy link
Copy Markdown
Contributor

As discussed in #1471, we already have the zoom buttons in the bottom right corner, so having those in the menu just adds clutter, instead of actually presenting new menu options.

New View menu, now without the redundant "Zoom In/Out" options

As discussed in #1471, we already have the zoom buttons in the bottom right corner, so having those in the menu just adds clutter, instead of actually presenting new menu options.
@yioannides

Copy link
Copy Markdown
Contributor

I was thinking how to rework the View panel menu, cause View icons aren't that common in raster editors, comparing to Edit / Effects / Adjustment etc, so I was thinking of something like that (pardon the crude mockup):

view-option

I was thinking whether we could have the View options right next to the zoom icon in the bottom of the screen, as it also pertains to zooming in/out and feels more compartmentalized (I went with the More... three-dot option icon).

@cameronwhite

Copy link
Copy Markdown
Member

One thing to be careful about is that the Show/Hide option lets you hide the status bar, so if the menu moves there you could end up in a state where you can't get the status bar back :)
That section, and the color scheme, also are more application-level settings that feel like they should be more in a "main" menu IMO

@pedropaulosuzuki

pedropaulosuzuki commented May 25, 2025

Copy link
Copy Markdown
Contributor Author

I was thinking whether we could have the View options right next to the zoom icon in the bottom of the screen, as it also pertains to zooming in/out and feels more compartmentalized (I went with the More... three-dot option icon).

I had that idea at the start, but the view menu shows more than just zoom levels (which we just removed half of the items in this Merge Request), so I believe it's more adequate to keep it at the top, as it is currently.

@cameronwhite cameronwhite merged commit 92ad39c into PintaProject:master May 27, 2025
6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants