Currently the language setting shows one language available in the system. If the user clicks the remove button, nothing happens. While this may be a good indication that the default language cannot be removed, it would be a stronger message if the user received a warning. For example, "The default language cannot be removed. Add more languages, select a new default and then remove this language." This could be as an in-kcm warming or maybe a popup instead. The user should receive more intentional information about why the remove action doesn't work.
Would it make sense to just disable the remove action?
It would make sense. Yeah. I would still include language showing the process somewhere in the window.
Yes, we should just hide the remove action when only one language is present.
A possibly relevant merge request was started @ https://invent.kde.org/plasma/plasma-workspace/-/merge_requests/892
Git commit c6c4c9adf528d2869db7708296fcb721b88d93e7 by Nate Graham. Committed on 28/05/2021 at 22:30. Pushed by ngraham into branch 'master'. [kcms/translations] Be more consistent with inline remove buttons To be consistent with what we generally do in other places, use the trashcan icon for the remove button, and only show it when the list has more than one item in it. Related: bug 437566 FIXED-IN: 5.22 M +2 -1 kcms/translations/package/contents/ui/main.qml https://invent.kde.org/plasma/plasma-workspace/commit/c6c4c9adf528d2869db7708296fcb721b88d93e7
Git commit b3a04fc0e172f779fc2e2eca9534996054eb922a by Nate Graham. Committed on 28/05/2021 at 22:30. Pushed by ngraham into branch 'Plasma/5.22'. [kcms/translations] Be more consistent with inline remove buttons To be consistent with what we generally do in other places, use the trashcan icon for the remove button, and only show it when the list has more than one item in it. Related: bug 437566 FIXED-IN: 5.22 (cherry picked from commit c6c4c9adf528d2869db7708296fcb721b88d93e7) M +2 -1 kcms/translations/package/contents/ui/main.qml https://invent.kde.org/plasma/plasma-workspace/commit/b3a04fc0e172f779fc2e2eca9534996054eb922a