nextcloud / forms

📝 Simple form & survey app for Nextcloud
https://apps.nextcloud.com/apps/forms
GNU Affero General Public License v3.0
324 stars 98 forks source link

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

Closed Chartman123 closed 6 months ago

Chartman123 commented 6 months ago

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 chris-hartmann@gmx.de

Chartman123 commented 6 months ago

/backport to stable4

Chartman123 commented 6 months ago

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...

susnux commented 6 months ago

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

Chartman123 commented 6 months ago

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

jotoeri commented 6 months ago

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:

susnux commented 6 months ago

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

I like the idea :+1:

Chartman123 commented 6 months ago

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

susnux commented 6 months ago

Would look like this: image

Or: image


Feel free to drop the commit.

Chartman123 commented 6 months ago

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.

susnux commented 6 months ago

image

or

image

Chartman123 commented 6 months ago

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?