-
-
Notifications
You must be signed in to change notification settings - Fork 1.2k
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
Bookmarks #37
Comments
I start working on it (using a left drawer atm) |
Thanks and exciting to know that! I'm assigning this to you. 🎉 @Grafikart One quick design suggestion. Can you please implement the bookmarks to be shown in the space below the address bar? As that is how most browsers do and I think, we should keep it that way. So that, it will be more intuitive for the users than adding it to the sidebar. What do you think? |
I don't know the project so I rely on existing elements. I don't feel confident enough to add a new UI element at the moment. I'm trying to get a first quick proof of concept. Then when I'll have more time i'll try to mirro the chrome user experience
|
@Grafikart We don't have to exactly mirror the chrome user experience. But only the placement if we could do it below the address bar it would be great. The code change should be almost similar, instead of adding it to the sidebar, you will have to add it to the NavigationControls component. |
Ok do you think we should ask the user for a title or use the domain name for the bookmark ? |
To start with I would suggest, we can just take the page title from <title
/> tag and use it.
If the title is missing we can fallback to the domain name.
…On Thu, 4 Jun 2020 at 11:03 PM, Jonathan ***@***.***> wrote:
Ok do you think we should ask the user for a title or use the domain name
for the bookmark ?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
<#37 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AAJZKYHHJSUOPGGSCZS6QATRU7LGBANCNFSM4IUACT7A>
.
|
No description provided.
The text was updated successfully, but these errors were encountered: