Bug 349084

Summary:	Items in View menu show inconsistent behaviour
Product:	[Applications] krita	Reporter:	Victor Wåhlström <victor.wahlstrom>
Component:	Usability	Assignee:	Krita Bugs <krita-bugs-null>
Status:	RESOLVED INTENTIONAL
Severity:	normal	CC:	halla
Priority:	NOR
Version First Reported In:	git master (please specify the git hash!)
Target Milestone:	---
Platform:	Other
OS:	Linux
Latest Commit:		Version Fixed/Implemented In:
Sentry Crash Report:

Description Victor Wåhlström 2015-06-13 02:14:31 UTC

Some actions affect only the current image/canvas, some affect all of them. Some are automatically toggled back of when you switch image/canvas, some are not.

View->Show Grid only affects the active image, and will remember the toggle on a per-image basis.

View->Show Rulers affect all images.

Suggestion:

View menu is a fairly global concept. It makes more sense if all actions under it are global, and not per-image. For any actions that should be set on a per-image basis, I propose moving them to Image->view or similar.

Such a split could also help improve code design.

Reproducible: Always

Comment 1 Halla Rempt 2015-06-13 03:10:51 UTC

Well, most of those things must be per-view -- you want one view on a certain image mirrored, and the other view not. Same with grid, rulers, zoom and so on. I haven't got a working Krita right now so I can't make an exhaustive list of what in the view menu should be per-view and what should be global.

Comment 2 Halla Rempt 2015-06-20 07:42:23 UTC

Okay, I'm home, so we can make a list of what should do what... I think that most users will be fine with per-window and per-canvas actions mixed in the view menu, because Photoshop does that as well... Moving the View Menu to the left of the Window menu would be a good change as well, I think.

View/show canvas only: per window
View/full screen mode: per window
View/Wrap around mode: per canvas, should move to View/Canvas
View/Canvas/mirror view: per canvas
View/Canvas/Reset Zoom: per canvas
View/Canvas/Rotate Canvas Right: per canvas
View/Canvas/Rotate Canvas Left: per canvas
View/Show Rulers: per canvas
View/Show Status Bar: per window
View/Show Grid: per canvas
View/Grid Spacing/*: per canvas
View/Show Perspective Grid: per canvas
View/Clear Perspective Grid: per canvas
View/Show Painting Assistants: per canvas
View/Show Assistant Previews: per canvas

Comment 3 Halla Rempt 2018-11-30 10:20:13 UTC

I don't think it's worth it to keep this open. People are used to the current menu layout and it's documented as it is. Nobody has been reporting they were confused about it either.