Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Subjects disappear when clicked in metadata editing #6339

Closed
kaitlinnewson opened this issue Nov 3, 2019 · 5 comments · Fixed by #6356
Closed

Subjects disappear when clicked in metadata editing #6339

kaitlinnewson opened this issue Nov 3, 2019 · 5 comments · Fixed by #6356

Comments

@kaitlinnewson
Copy link
Contributor

Tested in version 4.17 in both Harvard and Scholars Portal Dataverse instances. Tested in Chrome (78.0.3904.70) and Firefox (70.0) on MacOS Mojave.

When creating a dataset and entering metadata in Google Chrome, clicking on the name/label of the subject makes the subject box disappear. The issue does not happen if you only click the checkbox.

The problem starts at the "Earth and Environmental Science" subject, which causes the box to partially collapse. Clicking subjects below "Earth and Environmental Science" will cause the block to disappear completely.

Clicking on the text label for "Earth and...":

Screen Shot 2019-11-03 at 10 38 25 AM

Clicking on any text label below "Earth and...":

Screen Shot 2019-11-03 at 10 34 27 AM

The bug does not occur in Firefox; other browsers were not tested. Screenshots above were created from the Harvard instance.

@jggautier
Copy link
Contributor

jggautier commented Nov 3, 2019

Can confirm. The same thing happens with the Language field, when clicking on the words (not the checkboxes) of any of the listed options after the sixth option, starting with Arabic. "Earth and Environmental Science" is the seventh option in the Subjects field. Also happens with other check box fields with seven or more options.

@djbrooke
Copy link
Contributor

djbrooke commented Nov 4, 2019

Thanks @kaitlinnewson and @jggautier. We'll bring it into a sprint very soon, hopefully this week.

@sekmiller sekmiller self-assigned this Nov 6, 2019
@djbrooke djbrooke added the Small label Nov 6, 2019
sekmiller added a commit that referenced this issue Nov 8, 2019
@sekmiller sekmiller removed their assignment Nov 8, 2019
sekmiller added a commit that referenced this issue Nov 15, 2019
sekmiller added a commit that referenced this issue Nov 18, 2019
@mheppler
Copy link
Contributor

Saw this was reviewed, approved and moved to QA. There wasn't a review of the UI, so I jumped in and took a look. Here is a quick mockup of a proposed solution to use the same p:selectCheckboxMenu component that is used in editFilesFragment for the Edit Tags popup.

Can review with @TaniaSchlatter once we get a prototype to demo.

Screen Shot 2019-11-18 at 11 10 53 AM

@sekmiller
Copy link
Contributor

The reason that the original fix wasn't sent to the design team for review was because it didn't change the UI. The original change was to swap out the generic div scroll container to a primefaces scroll panel. This worked in most cases, except in Chrome where you used the scroller before clicking the item label.

@sekmiller
Copy link
Contributor

Changed to a select checkbox menu which solves the issue in Chrome(and works as expected in general). If @mheppler and @TaniaSchlatter want to take a look and see if it needs additional styling, please be my guest.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

5 participants