Created attachment 161557 [details] the category view from the viewer SUMMARY *** To successfully assign a token in the viewer, you need to use capital letters (caps lock). Otherwise, the token gets assigned as lower case letter, which does not show in the tokens list (but in the category list in viewer and thumbnail view). It is even possible to assign a lowercase and an uppercase token for the same letter "h" and "H". Then the token does not show in the tokens list. *** STEPS TO REPRODUCE 1. Start viewer and hit crt-t to allow assigning tokens 2. hit any letter, eg. "h" 3. The hint "Token H" appears and disappears and the letter "h" is listed in the category as token 4. exit the viewer and go to the start page 5. select Tokens OBSERVED RESULT * The newly assigned token is not in the list. * But in thumbnail view and viewer, the letter "h" is listed as token * When assigning "h" and "H" as tokens, both are listed in the categories in thumbnail view and viewer but "H" is not shown in the list of tokens. EXPECTED RESULT * The token should have been assigned as uppercase "H" regardless of caps lock setting and should always appear in the token list (the one reachable from the start page). That would be the old behaviour - see https://bugs.kde.org/show_bug.cgi?id=474307 and would be consistent with assigning tokens in the thumbnail view. SOFTWARE/OS VERSIONS KPA: v5.11.0-92-g28815029 Linux/KDE Plasma: KDE Plasma Version: KDE Frameworks Version: 5.103.0 Qt Version: 5.15.8 ADDITIONAL INFORMATION
Created attachment 161558 [details] Token list does not list the "H"
Additional effect/inconsistency: It is not possible to get rid of lowercase tokens in the thumbnail view or in the annotation dialogue.
Git commit d84d2d535fd32f9f95918b8716b20e6db8474757 by Johannes Zarl-Zierl. Committed on 11/09/2023 at 23:35. Pushed by johanneszarl into branch 'master'. Ensure that tokens are upper case. This was a regression introduced by commit 3a130ed45089c1b043bea8834a23be0438b4ee17. M +11 -7 Viewer/ViewerWidget.cpp M +1 -1 Viewer/ViewerWidget.h https://invent.kde.org/graphics/kphotoalbum/-/commit/d84d2d535fd32f9f95918b8716b20e6db8474757
Hi Andreas, Thanks for the report! I believe the regression is fixed by commit d84d2d53. Cheers, Johannes
As far as I can see, I can confirm that the issue is fixed with 5.11.0-100-gd84d2d53 :-) Eliminating the remaining lower case tokens took a bit of vim-ing but was OK.
Am Dienstag, 12. September 2023, 00:52:45 CEST schrieben Sie: > As far as I can see, I can confirm that the issue is fixed with > 5.11.0-100-gd84d2d53 :-) Thanks for checking! > Eliminating the remaining lower case tokens took a bit of vim-ing but was > OK. Sorry! Cheers, Johannes