View Issue Details

Related ChangesetsJump to NotesJump to History
This bug affects 1 person(s).
 6
IDProjectCategoryView StatusDate SubmittedLast Update
19299Feature requestsUsability/user experiencepublic2023-12-04 11:332024-01-18 18:26
Reporterf_funke Assigned ToDenisChenu  
PrioritynoneSeverityfeature 
Status resolvedResolutionfixed 
Summary19299: Unmask token field
Description

In closed-access surveys participants can enter the token manually. This field is currently input typ "password" so all characters are masked. From a user perspective - especially for survey participants who might be challenged with an online survey - it would be nice if the token field could be unmasked so that the code that has been entered would be visible.

Perhaps a technical solution would be to use something like this:
https://www.w3schools.com/howto/howto_js_toggle_password.asp

TagsNo tags attached.
Bug heat6
Story point estimate0
Users affected %0

Relationships

Relationship GraphDependency Graph
related to 19307 new Bug reports Ability to show password before login 

Users monitoring this issue

There are no users monitoring this issue.

Activities

alitsortp

alitsortp

2023-12-07 08:37

reporter   ~78901

That's right. Such hidden characters will cause many difficulties for survey participants.
DenisChenu

DenisChenu

2023-12-13 09:53

developer   ~78966

Even for password : allow show password it's a great UX solution !
DenisChenu

DenisChenu

2023-12-13 10:01

developer   ~78968

Maybe move this to an UX issue ?
Not a feature.
DenisChenu

DenisChenu

2023-12-19 17:22

developer   ~79049

@f_funke : ready for code review and testing (it's more visual than anything)
DenisChenu

DenisChenu

2023-12-21 10:42

developer   ~79058

https://github.com/LimeSurvey/LimeSurvey/pull/3673
DenisChenu

DenisChenu

2023-12-21 10:43

developer   ~79059

I think you work on Survey Theme :)
DenisChenu

DenisChenu

2024-01-18 18:24

developer   ~79221

Fix committed to master branch: http://bugs.limesurvey.org/plugin.php?page=Source/view&id=35975
tibor.pacalat

tibor.pacalat

2024-01-18 18:26

administrator   ~79222

Did code review, tested and merged.

Related Changesets

LimeSurvey: master d7578503

2024-01-18 18:24:36

DenisChenu


Committer: GitHub Details Diff 		New feature 19299: Unmask token field (03673) Affected Issues
19299
mod - themes/survey/fruity_twentythree/views/subviews/logincomponents/token.twig Diff File
mod - themes/survey/vanilla/views/subviews/logincomponents/token.twig Diff File

Issue History

Date Modified Username Field Change
2023-12-04 11:33 f_funke New Issue
2023-12-07 08:37 alitsortp Note Added: 78901
2023-12-07 08:37 alitsortp Bug heat 0 => 2
2023-12-13 09:53 DenisChenu Note Added: 78966
2023-12-13 09:53 DenisChenu Bug heat 2 => 4
2023-12-13 10:01 DenisChenu Relationship added related to 19307
2023-12-13 10:01 DenisChenu Note Added: 78968
2023-12-14 11:46 DenisChenu Assigned To => DenisChenu
2023-12-14 11:46 DenisChenu Status new => assigned
2023-12-19 17:22 DenisChenu Assigned To DenisChenu =>
2023-12-19 17:22 DenisChenu Status assigned => ready for code review
2023-12-19 17:22 DenisChenu Note Added: 79049
2023-12-21 10:42 DenisChenu Note Added: 79058
2023-12-21 10:43 DenisChenu Assigned To => p_teichmann
2023-12-21 10:43 DenisChenu Note Added: 79059
2024-01-18 17:36 DenisChenu Assigned To p_teichmann => gabrieljenik
2024-01-18 18:24 DenisChenu Changeset attached => LimeSurvey master d7578503
2024-01-18 18:24 DenisChenu Note Added: 79221
2024-01-18 18:26 tibor.pacalat Assigned To gabrieljenik => DenisChenu
2024-01-18 18:26 tibor.pacalat Status ready for code review => resolved
2024-01-18 18:26 tibor.pacalat Resolution open => fixed
2024-01-18 18:26 tibor.pacalat Note Added: 79222
2024-01-18 18:26 tibor.pacalat Bug heat 4 => 6