-
Notifications
You must be signed in to change notification settings - Fork 437
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
remove obsolete label element in metadata-schema.component.html #2505
Conversation
Thanks @saschaszott. If I understand correctly, by "find in page" you mean the Ctrl-F / Cmd-F in the browser? I tried searching for Did I understand correctly? |
@alanorth , the problem only arises if you search with a field name that contains two periods. This screenshot was taken from Google Chrome where a search with term |
@alanorth , I cannot reproduce this odd behaviour in Safari. This seems to be a Chrome specific problem. Nevertheless, it should be fixed as Chrome has > 50% market coverage. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This introduces trailing whitespace.
@saschaszott I finally looked at this. The existing code doesn't make sense—why put a label with only part of the metadata field? Could have been a bug in the original code. Anyways, I tested this and it doesn't break my find in Firefox so I'm +1. Would you feel comfortable rebasing this commit to remove the trailing whitespace? Thanks. |
Thank you for noticing - it was fixed. |
Thanks @saschaszott. I will squash the two commits into one before merging so the whitespace doesn't stay in git history. |
Successfully created backport PR for |
This PR implements a minor HTML change which improves the "find in page" of your prefered web browser. Currently, it is not possible to search for metadata field names of the form
schema.element.qualifier
using the "find in page" function, e.g. in /admin/registries/metadata/dc.