mirror of
https://github.com/advplyr/audiobookshelf-app.git
synced 2026-07-25 22:18:38 +02:00
[PR #1612] [MERGED] Update create bookmark to not autofocus input and use placeholder #1634
Reference in New Issue
Block a user
📋 Pull Request Information
Original PR: https://github.com/advplyr/audiobookshelf-app/pull/1612
Author: @advplyr
Created: 6/23/2025
Status: ✅ Merged
Merged: 6/23/2025
Merged by: @advplyr
Base:
master← Head:create_bookmarks_focus📝 Commits (1)
a77f345Update create bookmark to not autofocus input and leave empty with placeholder📊 Changes
3 files changed (+18 additions, -15 deletions)
View changed files
📝
components/modals/BookmarksModal.vue(+6 -12)📝
components/ui/TextInput.vue(+5 -1)📝
components/ui/TextInputWithLabel.vue(+7 -2)📄 Description
Brief summary
When pressing create bookmark the current functionality is to focus the input and select the input value.
This update removes the autofocus and leaves the input value blank with a placeholder value.
Which issue is fixed?
No issue but fixes issue mentioned in PR #1611
Pull Request Type
Both android/ios
In-depth Description
Auto-focusing and selecting the bookmark input was originally added because when using a custom bookmark note the input value had to be deleted first. By having the input text auto-selected you could start typing for a custom note or press create to use the default note.
This PR removes the need to auto-select the text in the input because the input will be empty. The only downside is the input will have to be pressed to open the keyboard to add a custom note instead of automatically opened.
If it is preferred to auto open the keyboard we could look at a different design that uses a quick create button or uses a similar design to the web app.
Screenshots
🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.