forms icon indicating copy to clipboard operation
forms copied to clipboard

fix: add icon to title of required questions in edit mode

Open Chartman123 opened this issue 1 year ago • 4 comments

This fixes #2098 by adding a visual indicator in edit mode that a question is required:

grafik

(Also removes some now obsolete conditions)

Signed-off-by: Christian Hartmann [email protected]

Chartman123 avatar Apr 25 '24 21:04 Chartman123

/backport to stable4

Chartman123 avatar Apr 29 '24 06:04 Chartman123

What do you think instead about a little red circle overlaid on the 3-dot menu (much like when you have unread notifications), which calls attention to opening the menu which then also explains that it is required?

@jancborchardt thanks for the review. I think if we want to point out directly that one of settings of the submenu is enabled, it should be directly visible. Otherwise you wouldn't know what the red ring means and you'd have to look into the menu.

But I'm with you that it is probably a little bit too big...

Chartman123 avatar Apr 29 '24 16:04 Chartman123

Maybe just add it to the placeholder? So instead of "People can enter a number" -> "People can enter a number (required)" or similar?

susnux avatar Apr 29 '24 16:04 susnux

For text questions ok, but how would you do it for e.g. the new file question?

Chartman123 avatar Apr 29 '24 16:04 Chartman123

Was just dropping by so here some greets and my 5 cent: I like the mark on the three dot menu - but i'd think about using an asterisk there. This should be clearly connected to the required option. I could imagine one could do that similarly with specific signs of up to three options from the menu (or maybe even four on the four edges)? :wink:

:wave:

jotoeri avatar May 01 '24 10:05 jotoeri

I like the mark on the three dot menu - but i'd think about using an asterisk there.

I like the idea :+1:

susnux avatar May 06 '24 15:05 susnux

So we overlay the Icon in small over the three dot menu?

Chartman123 avatar May 06 '24 19:05 Chartman123

Would look like this: image

Or: image


Feel free to drop the commit.

susnux avatar May 07 '24 13:05 susnux

I could live with that, but move it to the top left and don't use the red color. I think this is too much.

Chartman123 avatar May 07 '24 13:05 Chartman123

image

or

image

susnux avatar May 08 '24 09:05 susnux

Thanks for updating the screenshots and the code :) I only have little time right now...

I'd be fine with that second version without the accent color :) @jancborchardt what do you think?

Chartman123 avatar May 08 '24 11:05 Chartman123