[krita] [Bug 439699] Tag is reset to nothing on adding or removing a storage
https://bugs.kde.org/show_bug.cgi?id=439699 Emmet O'Neill changed: What|Removed |Added Resolution|--- |FIXED Status|CONFIRMED |RESOLVED -- You are receiving this mail because: You are watching all bug changes.
[krita] [Bug 439699] Tag is reset to nothing on adding or removing a storage
https://bugs.kde.org/show_bug.cgi?id=439699 --- Comment #11 from Eoin O'Neill --- Git commit 270ceaa330569bd43ce27d55d118794602bd3fbe by Eoin O'Neill. Committed on 02/08/2021 at 23:14. Pushed by eoinoneill into branch 'master'. Modify how resource model is updated when adding / removing storage. Restored tradition model reset behavior in favor of responding to changes within the KisTagChooserWidget class itself. This should be more sane... M +4-4libs/resources/KisTagModel.cpp M +21 -1libs/resourcewidgets/KisTagChooserWidget.cpp M +11 -0libs/resourcewidgets/KisTagChooserWidget.h https://invent.kde.org/graphics/krita/commit/270ceaa330569bd43ce27d55d118794602bd3fbe -- You are receiving this mail because: You are watching all bug changes.
[krita] [Bug 439699] Tag is reset to nothing on adding or removing a storage
https://bugs.kde.org/show_bug.cgi?id=439699 --- Comment #10 from Tiar --- Reopened, even though it probably works most of the time; but explanation written here: https://invent.kde.org/graphics/krita/-/commit/2317edd3b526eecf09487a6476f4e4aa39365c95#note_281331 - basically it only works if the storage added or removed didn't add or remove any tags. -- You are receiving this mail because: You are watching all bug changes.
[krita] [Bug 439699] Tag is reset to nothing on adding or removing a storage
https://bugs.kde.org/show_bug.cgi?id=439699 --- Comment #9 from Emmet O'Neill --- The commit above has addressed this issue on my end, I think. @Tiar: Because Eoin and I were seeing something different than you on both of our machines, you might want to check and see if the issue that you were having has also been fixed. If so, please close this bug. :) -- You are receiving this mail because: You are watching all bug changes.
[krita] [Bug 439699] Tag is reset to nothing on adding or removing a storage
https://bugs.kde.org/show_bug.cgi?id=439699 --- Comment #8 from Emmet O'Neill --- Git commit 2317edd3b526eecf09487a6476f4e4aa39365c95 by Emmet O'Neill. Committed on 30/07/2021 at 21:57. Pushed by emmetoneill into branch 'master'. Resource Management: Corrected TagFilter Combobox Behavior. The combobox for selecting tags within resource chooser widgets no longer resets when adding and removing storage bundles. (Thanks for the help Eoin.) M +4-4libs/resources/KisTagModel.cpp M +13 -0libs/resources/KisTagResourceModel.cpp M +4-1libs/resources/KisTagResourceModel.h https://invent.kde.org/graphics/krita/commit/2317edd3b526eecf09487a6476f4e4aa39365c95 -- You are receiving this mail because: You are watching all bug changes.
[krita] [Bug 439699] Tag is reset to nothing on adding or removing a storage
https://bugs.kde.org/show_bug.cgi?id=439699 Emmet O'Neill changed: What|Removed |Added Status|REOPENED|CONFIRMED --- Comment #7 from Emmet O'Neill --- So I've been making some progress on this bug, but there are a few problems. First off, I've been seeing what Halla has been where the combobox is never blank, and resets to the all tag when changing active storages. This reset to all shouldn't happen though, imo, and I've begun fixing that particular issue. So far, I've been able to get the selected tag to persist across resource changes (good) but I have run into a problem where the resources that belong to a tag don't accurately reflect active resources. I was able to fix a related issue to this w/ bug 440315 where the TagResourceModel wouldn't update on any of the drop down options until a tag was assigned or removed. So I might be able to go about it the same way... -- You are receiving this mail because: You are watching all bug changes.
[krita] [Bug 439699] Tag is reset to nothing on adding or removing a storage
https://bugs.kde.org/show_bug.cgi?id=439699 Emmet O'Neill changed: What|Removed |Added Resolution|FIXED |--- Status|RESOLVED|REOPENED --- Comment #6 from Emmet O'Neill --- Err.. Tagged the wrong bug. -- You are receiving this mail because: You are watching all bug changes.
[krita] [Bug 439699] Tag is reset to nothing on adding or removing a storage
https://bugs.kde.org/show_bug.cgi?id=439699 Emmet O'Neill changed: What|Removed |Added Resolution|--- |FIXED Status|CONFIRMED |RESOLVED Latest Commit||https://invent.kde.org/grap ||hics/krita/commit/a51f7ee07 ||8c006eb601883529373b0c1dccc ||cdb5 --- Comment #5 from Emmet O'Neill --- Git commit a51f7ee078c006eb601883529373b0c1ddb5 by Emmet O'Neill. Committed on 30/07/2021 at 00:42. Pushed by emmetoneill into branch 'master'. Bugfix for resource chooser tag filters not reflecting active storage changes. Tag dropdown boxes will now always correctly reflect active / inactive resource storages. M +6-4libs/resources/KisTagFilterResourceProxyModel.cpp M +20 -0libs/resources/KisTagResourceModel.cpp M +4-0libs/resources/KisTagResourceModel.h M +1-1libs/resourcewidgets/KisTagChooserWidget.h https://invent.kde.org/graphics/krita/commit/a51f7ee078c006eb601883529373b0c1ddb5 -- You are receiving this mail because: You are watching all bug changes.
[krita] [Bug 439699] Tag is reset to nothing on adding or removing a storage
https://bugs.kde.org/show_bug.cgi?id=439699 --- Comment #4 from Tiar --- Created attachment 140358 --> https://bugs.kde.org/attachment.cgi?id=140358=edit Screenshot showing lack of text (selected tag) in the tag chooser combobox Oh I just realized what is the confusion. It's not that the combobox has no options; no, the combobox does have all the options, hopefully even correct (respecting deactivated storages). The problem is that the *selected tag* on the combobox is empty, which means that it looks empty if you don't click on it. I believe the filtering is "All", but the text is missing. It's the standard "nothing is selected if you reset the model" situation that we had in resource choosers before (a year ago, before Halla's big changes to models to avoid using model reset). I attached a screenshot showing the problem. -- You are receiving this mail because: You are watching all bug changes.
[krita] [Bug 439699] Tag is reset to nothing on adding or removing a storage
https://bugs.kde.org/show_bug.cgi?id=439699 --- Comment #3 from Tiar --- > I don't see an empty tag combobox if I enable/disable a storage in the > storage selctor widget? That's interesting, I still see it. I opened Krita fresh on a3e9ff58d0, and on startup, tag comboboxes in any resource chooser is empty. Then I select, let's say, "All" tag. It shows up, as it should. Then I click on the storage enabling popup, and let's say enable and disable the Krita 3 bundle. The tag disappears from the combobox. -- You are receiving this mail because: You are watching all bug changes.
[krita] [Bug 439699] Tag is reset to nothing on adding or removing a storage
https://bugs.kde.org/show_bug.cgi?id=439699 Halla Rempt changed: What|Removed |Added CC||ha...@valdyas.org --- Comment #2 from Halla Rempt --- I don't see an empty tag combobox if I enable/disable a storage in the storage selctor widget? -- You are receiving this mail because: You are watching all bug changes.
[krita] [Bug 439699] Tag is reset to nothing on adding or removing a storage
https://bugs.kde.org/show_bug.cgi?id=439699 Emmet O'Neill changed: What|Removed |Added Assignee|krita-bugs-n...@kde.org |emmetoneill@gmail.com CC||emmetoneill@gmail.com -- You are receiving this mail because: You are watching all bug changes.
[krita] [Bug 439699] Tag is reset to nothing on adding or removing a storage
https://bugs.kde.org/show_bug.cgi?id=439699 Halla Rempt changed: What|Removed |Added Status|REPORTED|CONFIRMED Ever confirmed|0 |1 -- You are receiving this mail because: You are watching all bug changes.
[krita] [Bug 439699] Tag is reset to nothing on adding or removing a storage
https://bugs.kde.org/show_bug.cgi?id=439699 Tiar changed: What|Removed |Added Keywords||regression, release_blocker --- Comment #1 from Tiar --- WHAT NEEDS TO BE DONE: This is caused by: ''' beginResetModel(); resetQuery(); endResetModel(); ''' in both "removeStorage" and "addStorage". There are two ways to fix it. 1) "Proper" way: avoid using resetModel(), and instead use rowsAdded(), rowsRemoved(). Con: possibly error-prone if the developer uses some assumptions about the order of tags that turn out to be untrue. 2) Implement it in every tag chooser to "remember" the last used tag and set it after every resetModel. NOTE: This issue is related to the issue of remembering the tag after closing Krita. It would be best if every resource chooser had its own place in configuration to remember its own tag. It is especially tricky for tags in the brush chooser popup from the toolbar and the brush chooser in Brush Presets docker. -- You are receiving this mail because: You are watching all bug changes.
