ecamp3 icon indicating copy to clipboard operation
ecamp3 copied to clipboard

Fix #3499

Open MarcoAllenspach opened this issue 2 years ago • 4 comments

fixed #3499

image

Remove sentence but not hint

MarcoAllenspach avatar Jan 08 '24 18:01 MarcoAllenspach

Feature branch deployment currently inactive.

If the PR is still open, you can add the deploy! label to this PR to trigger a feature branch deployment.

github-actions[bot] avatar Jan 09 '24 15:01 github-actions[bot]

@manuelmeister Like this? image

I don't think this is better because the user can press the button and don't know why it doesn't works if he don't read carefully... Or not?

MarcoAllenspach avatar Jan 09 '24 21:01 MarcoAllenspach

@manuelmeister see my last comment

MarcoAllenspach avatar Jan 18 '24 17:01 MarcoAllenspach

@MarcoAllenspach I recommend removing the tooltip around the readonlyRole select and instead readd the hint to the select. I think it is still good to remove the hint on the non-readonlyRole select, as it is only relevant if you edit the last manager. This approach provides a clearer user experience, as it directly explains within the textfield why it is not editable, rather than relying on a tooltip.

Regarding the tooltip on the button, I suggest keeping it. Apologies for any confusion caused by my previous response.

manuelmeister avatar Jan 19 '24 07:01 manuelmeister

If @MarcoAllenspach or someone else wants to continue, they are free to reopen it. The proposal is also referenced in the issue.

BacLuc avatar Apr 14 '24 10:04 BacLuc

@manuelmeister Sorry was really busy the last time I recommend removing the tooltip around the readonlyRole select and instead readd the hint to the select. I think it is still good to remove the hint on the non-readonlyRole select, as it is only relevant if you edit the last manager. This approach provides a clearer user experience, as it directly explains within the textfield why it is not editable, rather than relying on a tooltip.

So its like in my picture on the comment from January 9te?

Or do i understand it wrong?

MarcoAllenspach avatar Jun 03 '24 16:06 MarcoAllenspach