Currently the fieldset is collapsible/expanded. Instead start with the fieldset collapsed. There is no need to bother the use with similar content if there isn't any (i.e. entering a unique content). Expand the fieldset when there is similar content to display. Only do this once so that if they've manually collapsed it, the widget doesn't fight them.

For my use case, this is pretty important to a good UX.

CommentFileSizeAuthor
#3 uniqueness-autopen_fieldset-1893190-3.patch2.6 KBdanchadwick
#1 uniqueness-autopen_fieldset-1893190-1.patch1.79 KBdanchadwick

Comments

danchadwick’s picture

danchadwick commented
Status: Active » Needs review
StatusFileSize
new uniqueness-autopen_fieldset-1893190-1.patch1.79 KB

Due to overall indentation in jQuery Initialization, this patch must be applied second.

bforchhammer’s picture

bforchhammer commented
Status: Needs review » Needs work

Change makes perfect sense to me. Patch needs a reroll, with #1892868: jQuery code not protected from multiple initialization by behavior having been committed.

danchadwick’s picture

danchadwick commented
Status: Needs work » Needs review
StatusFileSize
new uniqueness-autopen_fieldset-1893190-3.patch2.6 KB

There was an indentation error introduced in the else clause in the update function. I suspect this is why the patch wouldn't apply.

This re-roll fixes that indentation and re-applies the patch.

danchadwick’s picture

danchadwick commented

And because that indentation affects the line of code changed in the unpublished content issue, this patch should be applied before that one, which may need a re-roll (we'll have to wait and see).

bforchhammer’s picture

bforchhammer commented
Status: Needs review » Fixed

Thanks. I ended up fixing the indentation separately and committed #1.

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.