| Summary: | Advanced color selector do not allow to show Shade selector only | ||
|---|---|---|---|
| Product: | [Applications] krita | Reporter: | Camille Bissuel <welcome> |
| Component: | Dockers | Assignee: | Krita Bugs <krita-bugs-null> |
| Status: | RESOLVED FIXED | ||
| Severity: | wishlist | CC: | code, halla |
| Priority: | NOR | ||
| Version: | git master (please specify the git hash!) | ||
| Target Milestone: | --- | ||
| Platform: | unspecified | ||
| OS: | Linux | ||
| Latest Commit: | https://commits.kde.org/krita/52afd0cbfc8e23d1b4c9f519356dbe847fc2a890 | Version Fixed In: | |
| Attachments: | shade selector patch | ||
|
Description
Camille Bissuel
2014-07-10 15:14:46 UTC
Any hope to see this solved in 2.9 ? A simple "do not show" radio button in the "Color selector" tab of the options of the "advanced color selector" docker would do the trick ;) Thanks again for your great work on Krita ! Argh... I'm sorry, but this sliped through, and right now we're in string freeze so we cannot add a single checkbox before we release :-( My fault, I should have updated sooner... But, no worries, it's not urgent : a shortcut do the job right now... I just would prefer to have it in a docker. Maybe for an 2.9.X version, or something like that ;) Thanks again ! Hi Camille, If I'm not wrong, there needs to be a checkbox in Settings > Configure Krita > Color Selector Settings > Advanced Color Selector Docker Settings to turn on or off the color selector (the big triangle encompassed by a ring) in the Advanced Color Selector window on the right. Am I right? (Also, please pardon my lack of sense of naming components; I'm a newbie over here :) ). I'll try and fix it then. Yes, Ranveer, that's it, thanks for your proposal ! You can access the same settings by clicking the "spanner" icon in the docker itself. But the idea is to be able to hide this big triangle encompassed by a ring to display only the shade selector. Is it a problem if the window become fully empty (if the shade selector is also hidden)... ? I don't think so, but if it's a yes, hiding one should automatically show the other... (In reply to Camille Bissuel from comment #5) > Is it a problem if the window become fully empty (if the shade selector is > also hidden)... ? > I don't think so, but if it's a yes, hiding one should automatically show > the other... I didn't get you. Do you intend to hide the whole docker? AFAIK, the advanced color selector is composed of the color selector and the shade selector only. Hiding both would mean empty-ing the whole docker. @Boud So I was wondering how to do this.
I tracked down the file that renders the color chooser as "<path-to-src>/src/calligra/krita/plugins/extensions/dockers/advancedcolorselector/kis_color_selector_ng_docker_widget.cpp"
There, `m_colorSelectorContainer` is an object of the class `KisColorSelectorContainer` that controls the color selector (I removed all occurrences of this object in the file to see if this was indeed the case; It did work). So, I'll be storing a boolean value in `KGlobal::config()->group("advancedColorSelector")` that'll pertain to the visibility of the color selector.
What would be the best way to hide it though? Also, is my approach correct?
Hi Ranveer, I'll try to be more clear : even if the Color selector (2nd tab) and the Sshade selector (3rd tab) are hidden, there may be the "Color patches" (4th tab) left. So one may use this docker only to keep track of color history for example. So hiding both Color selector and Shade selector should ideally be possible, even if the docker look empty temporary ;) (In reply to Camille Bissuel from comment #8) > Hi Ranveer, > > I'll try to be more clear : even if the Color selector (2nd tab) and the > Sshade selector (3rd tab) are hidden, there may be the "Color patches" (4th > tab) left. > So one may use this docker only to keep track of color history for example. > Ah! I get it now. > So hiding both Color selector and Shade selector should ideally be possible, > even if the docker look empty temporary ;) Yes, it is definitely possible, as far as I see in the source code, both components are getting placed in the docker in a similar way. I'll try and implement it and send you a screenshot of the result to confirm, just in case. Git commit cb207e27e7cda184f0901e4ea5bcb79cb8bcf666 by Boudewijn Rempt. Committed on 19/02/2015 at 18:17. Pushed by rempt into branch 'master'. REVIEW:122638 CCMAIL:ranveeraggarwal@gmail.com Patch by Ranveer Aggarwal. Thanks! M +6 -0 krita/plugins/extensions/dockers/advancedcolorselector/kis_color_selector_container.cpp M +1 -0 krita/plugins/extensions/dockers/advancedcolorselector/kis_color_selector_container.h M +6 -0 krita/plugins/extensions/dockers/advancedcolorselector/kis_color_selector_settings.cpp M +7 -0 krita/plugins/extensions/dockers/advancedcolorselector/wdg_color_selector_settings.ui http://commits.kde.org/calligra/cb207e27e7cda184f0901e4ea5bcb79cb8bcf666 Great ! Thanks Ranveer ! Now I'm not only waiting 2.9, but 2.9.1 too ;) I'll try to compile from master in the meantime to give my feedback ;) Mmh... can't see anything compiling from AUR (https://aur.archlinux.org/packages/krita-git/) neither from the latest 2.9.1 packages... (https://www.archlinux.org/packages/extra/x86_64/calligra-krita/) Is this on a special branch ? Thanks ! Camille I'm sorry if I'm dumb... but it's now 2.9.4, and I still can't find this option in Krita... This patch was sent to master... but will it be included in the 2.9 releases ? Or some issue prevent it ? Sorry to disturb you with a resolved wishlist bug... There are other patches in master that I probably should merge to 2.9, since it's going to take a long time for master to reach users :-( This is one of them, so thanks for reminding me! Thanks Boudewijn.... ! I just understood today that 'master' branch was separated from '2.9' (and releases in general)... Making users and developers understand each other is still difficult... despite the Krita team is the most open minded and welcoming I ever met ;) I use the starting Bug hunting week to say again this is waiting for inclusion ;) Maybe a special tag in the bug tracker is needed for such waiting patches ? What do you suggest ? Thanks a lot again ! Created attachment 93247 [details]
shade selector patch
This is the patch; it got broken in the meantime and doesn't apply to 2.9 anymore, so it needs reworking.
Sadly reopening... Hope it's not too much work to make it work again... WISHGROUP: Small Usability Fixes Git commit 52afd0cbfc8e23d1b4c9f519356dbe847fc2a890 by Anna Medonosova. Committed on 01/01/2019 at 12:58. Pushed by amedonosova into branch 'master'. Advanced Color Selector: option to hide color selector Summary: I have resurrected the original patch by Ranveer Aggarwal, which enables the user to hide the color selector part of the docker. There was one issue with the original: the docker settings button is inside the color selector, therefore when you hide the selector, the only way to configure the docker is via the main menu, "Settings" -> "Configure Krita". To solve that I have added a fallback settings button to the docker widget, which appears when the color selector part is hidden. Reviewers: #krita, rempt Reviewed By: #krita, rempt Subscribers: rempt Tags: #krita Differential Revision: https://phabricator.kde.org/D17893 M +15 -8 plugins/dockers/advancedcolorselector/kis_color_selector_container.cpp M +1 -0 plugins/dockers/advancedcolorselector/kis_color_selector_container.h M +68 -4 plugins/dockers/advancedcolorselector/kis_color_selector_ng_docker_widget.cpp M +8 -0 plugins/dockers/advancedcolorselector/kis_color_selector_ng_docker_widget.h M +3 -3 plugins/dockers/advancedcolorselector/kis_color_selector_settings.cpp M +256 -244 plugins/dockers/advancedcolorselector/wdg_color_selector_settings.ui https://commits.kde.org/krita/52afd0cbfc8e23d1b4c9f519356dbe847fc2a890 Thanks a lot Anna ! and Happy new year ;) Hopefully this will land successfully into next release ;) |