-
Notifications
You must be signed in to change notification settings - Fork 21
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
Add option safari (and firefox?) #1750
Conversation
Deployed to https://pr-1750.aam-digital.net/ |
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.
Couldn't test for the target devices but code looks good 👍
@@ -142,6 +143,7 @@ export class BasicAutocompleteComponent<O, V = O> | |||
let filteredOptions = this._options; | |||
this.showAddOption = false; | |||
clearTimeout(this.addOptionTimeout); | |||
setTimeout(() => (this.showAddOption = false), 100); |
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.
observing actual users add options, I am tending towards removing the delay for new options again. It was complex enough for a non-savvy user to figure out they can do this without things moving and changing. Maybe instead highlight the "Add new" option with different color a bit more than already?
Sorry, somewhat off-topic here actually. Just popped into my mind.
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.
Got point. I removed the timeout and instead visually highlighted the add option text. Please have a look
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.
Nice. I like it.
In terms of UI logic, maybe we should use the blue accent color though (similar to add buttons above lists, etc.)?
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.
I get the point but it does not look good with the blue color.
Kudos, SonarCloud Quality Gate passed! 0 Bugs No Coverage information |
🎉 This PR is included in version 3.19.0 🎉 The release is available on GitHub release Your semantic-release bot 📦🚀 |
Visible/Frontend Changes
Architectural/Backend Changes
--