389033 2018-01-16 01:54:19 +0000 "Show Pixel Grid" checkbox state can desync with multiple documents/views 2018-01-30 08:21:51 +0000 1 1 2 Applications krita Usability git master (please specify the git hash!) Compiled Sources Linux RESOLVED FIXED NOR normal --- 1 nicholaslapointe8 scottpetrovic dimula73 freebox64 halla https://commits.kde.org/krita/95b2c02b0bac26cf87788a4ef31c8640635204cf 0 oldest_to_newest 1722460 0 nicholaslapointe8 2018-01-16 01:54:19 +0000 The "checked" state of the "Show Pixel Grid" checkbox inside of the "View" menu can be inaccurate when it is toggled in multiple documents or views. (Steps assume that the pixel grid is already enabled, which it is by default) 1. Create two documents 2. Disable the pixel grid in one document 3. Switch documents. The pixel grid in this document will still be enabled, but the checkbox is now unchecked. I'd expect that the state of the checkbox would match the status of the pixel grid on the view (versus the other alternative of the checkbox affecting all views). 1722507 1 freebox64 2018-01-16 10:44:41 +0000 Yes. I can confirm on Ubuntu 14.04 with Krita 4.0.0-beta1.1 and OpenGL enabled. 1722877 2 halla 2018-01-17 11:04:29 +0000 Probably caused by commit cb1e38118be3e986a252210fa8df61a8fc2d4b18 Author: Scott Petrovic <scottpetrovic@gmail.com> Date: Sat Oct 28 09:44:18 2017 -0500 turned the show pixel grid into an action and moved it to the view menu 1722967 3 scottpetrovic 2018-01-17 14:25:41 +0000 The pixel grid is on/off based on the kritarc currently. It only currently works globally, not per view. We will have to code something up so it keeps track of the enabled status by view and remove it from being on/off in the kritarc. 1722989 4 halla 2018-01-17 15:18:36 +0000 > The pixel grid is on/off based on the kritarc currently. It only currently > works globally, not per view. The weird thing is that there is some per-view code, or so it seems. I think that's not necessary, actually. 1723470 5 nicholaslapointe8 2018-01-18 23:11:02 +0000 I could imagine per-view control of it being helpful when a user has multiple views of the same document at different zoom levels, but I don't know how common such a use case would be. 1723578 6 scottpetrovic 2018-01-19 13:44:54 +0000 Git commit 1a47ae24712377821994c3b0309b9437da129d73 by Scott Petrovic. Committed on 19/01/2018 at 13:44. Pushed by scottpetrovic into branch 'master'. M +0 -5 libs/ui/KisViewManager.cpp M +0 -16 libs/ui/canvas/kis_canvas_controller.cpp M +0 -1 libs/ui/canvas/kis_canvas_controller.h M +11 -0 libs/ui/kis_paintop_box.cc M +3 -0 libs/ui/kis_paintop_box.h M +4 -3 libs/ui/opengl/kis_opengl_canvas2.cpp https://commits.kde.org/krita/1a47ae24712377821994c3b0309b9437da129d73 1723580 7 scottpetrovic 2018-01-19 13:48:36 +0000 I fixed the bug with it syncing. For now the pixel grid is going to have to be a global option and not per view. That was how it was coded and the way it is saved/loaded with the kritarc. It is really a settings option that wasn't designed to be in the main menu. If we start getting requests for people needing it differently per view, that can be a future feature request. 1725586 8 dimula73 2018-01-29 07:57:26 +0000 Hi, Scott! There is still a bug with sync'ness: When switching the state of the checkbox in one view, the **state of the checkbox** in another view goes out of sync :( 1725591 9 dimula73 2018-01-29 08:28:57 +0000 Added a review request: https://phabricator.kde.org/D10171 1725828 10 dimula73 2018-01-30 08:21:51 +0000 Git commit 95b2c02b0bac26cf87788a4ef31c8640635204cf by Dmitry Kazakov. Committed on 29/01/2018 at 08:27. Pushed by dkazakov into branch 'master'. Fix syncing pixel grid state between windows once again Summary: This patch extends commit1a47ae24712377821994c3b0309b9437da129d73 and addresses the following issues: 1) The state of the menu action checkbox was not updated when the other window changed the its state 2) There should be **no** access to KisConfig in the OpenGL rendering loop (it is slow due to possible disk access). 3) A canvas update call should be issued right after the config changed, otherwise the old state of the grid will be visible until one hovers the canvas Test Plan: 1) Open multiple Krita windows and zoom both of them to 1000% 2) Change View->Show Pixel Grid in one of the windows 3) The state on the checkbox and the canvas should automatically update in the other window Reviewers: #krita, scottpetrovic Differential Revision: https://phabricator.kde.org/D10171 M +1 -0 libs/ui/KisMainWindow.cpp M +16 -0 libs/ui/KisViewManager.cpp M +1 -0 libs/ui/KisViewManager.h M +9 -0 libs/ui/canvas/kis_canvas_controller.cpp M +1 -0 libs/ui/canvas/kis_canvas_controller.h M +5 -0 libs/ui/kis_config_notifier.cpp M +2 -0 libs/ui/kis_config_notifier.h M +0 -11 libs/ui/kis_paintop_box.cc M +0 -3 libs/ui/kis_paintop_box.h M +15 -6 libs/ui/opengl/kis_opengl_canvas2.cpp M +1 -0 libs/ui/opengl/kis_opengl_canvas2.h https://commits.kde.org/krita/95b2c02b0bac26cf87788a4ef31c8640635204cf