Page MenuHome GnuPG
Create Task
Maniphest T5857

Kleopatra: Change "List smartcard readers" to "select smartcard reader"
Closed, ResolvedPublic
Actions

Description

Under Settings we now have "List smartcard readers"
But to change the setting based on this listing a user has to discover that the setting is under "Kleopatra Settings -> GnuPG System -> Smartcard". This is not so nice when we could in theory just use a radio button next to the list of smartcard readers to offer such a selection.

The second Problem that this has is that many customers want us to hide the "GnuPG System" settings as they are too low level for most users. And when we hide it they cannot change the smartcard reader which can be important as we have at least one customer who does not use standardized readers and where each user would need to change the setting manually.

Revisions and Commits

rLIBKLEO Libkleo
rLIBKLEOb32aca3bef27 Extract the reader port combo box from the config entry UI
rLIBKLEO172385cf980d Set placeholder text instead of an editable default text
rKLEOPATRA Kleopatra
rKLEOPATRAe038551b4f14 Remove command to list the available smart card readers
rKLEOPATRA7c59b266e17f Add config module for smart card related settings

Event Timeline

aheinecke triaged this task as Wishlist priority.Feb 24 2022, 9:35 AM
aheinecke created this task.
ikloecker claimed this task.Feb 24 2022, 10:05 AM
ikloecker moved this task from Restricted Project Column to Restricted Project Column on the Restricted Project board.
ikloecker added a subscriber: ikloecker.

I have an uncommitted SmartCardConfigurationPage. I guess, I'll simply commit this and remove the "List smartcard readers" option.

• werner added a subscriber: • werner.Feb 24 2022, 12:36 PM

aheinecke: Good idea

ikloecker added a commit: rLIBKLEO172385cf980d: Set placeholder text instead of an editable default text.Feb 24 2022, 1:01 PM
ikloecker added a commit: rLIBKLEOb32aca3bef27: Extract the reader port combo box from the config entry UI.
ikloecker added a commit: rKLEOPATRA7c59b266e17f: Add config module for smart card related settings.Feb 24 2022, 1:06 PM
ikloecker added a comment.Feb 24 2022, 2:52 PM

There is now a dedicated configuration module for smart card related settings. Currently, it's rather empty, but maybe there are more smart card settings you want to see there.

The empty entry at the end can be filled with arbitrary values. It also helps to keep the currently configured value even if the corresponding reader isn't plugged in.

I propose to remove the rather useless "List Smartcard Readers" functionality, or, if you want to keep it as debugging helper, move it to a less prominent location.

aheinecke closed this task as Resolved.Feb 24 2022, 3:21 PM

Removing the list seems reasonable to me, we can tell users in support that they should go to settings- > Smartcard to select the reader used.

I would have preferred a small dialog with the setting, but having this as a config module is fine, too. There might be more smartcard related options in the future.
We might even add a button in there to start the TIM if it is available and installed so that users can initialize their smartcard in a workflow that starts through kleopatra. But this solves the problem now so -> resolved.

ikloecker added a commit: rKLEOPATRAe038551b4f14: Remove command to list the available smart card readers.Feb 24 2022, 4:38 PM