Issue 4440: Merge & Unify right-click lists on toolbar buttons
Posted: 04 Nov 2018, 16:04
So I decided to pick up the issue.
So, in accordance to "How to contribute" instruction, I forked the repo, created branch for the issue and this thread on forum. Then I moved both mPanel and mTable along with showPanel method implementation from SceneToolTextureBrush and SceneToolRun to their base class SceneTool. I'm not sure this is how it should be done, though. As far as I can see SceneToolMode, SceneToolToggle, SceneToolToggle2 are also subclasses of SceneTool. And it seems they don't have to show such behavior (show the panel on RMB press).
Am I doing things right? Any hints? Thanks.
So, in accordance to "How to contribute" instruction, I forked the repo, created branch for the issue and this thread on forum. Then I moved both mPanel and mTable along with showPanel method implementation from SceneToolTextureBrush and SceneToolRun to their base class SceneTool. I'm not sure this is how it should be done, though. As far as I can see SceneToolMode, SceneToolToggle, SceneToolToggle2 are also subclasses of SceneTool. And it seems they don't have to show such behavior (show the panel on RMB press).
Am I doing things right? Any hints? Thanks.