-
Notifications
You must be signed in to change notification settings - Fork 29
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
Create tabbed-mode-explainer.md #55
Conversation
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.
As the author of the Tabbed Application Mode article, I'm really excited about progress in this area. I have left a couple of comments and suggestions.
Co-authored-by: Thomas Steiner <tomac@google.com>
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.
Looks good to me!
Co-authored-by: Thomas Steiner <tomac@google.com>
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.
LGTM with nit
The proposed format makes it easy to add more fields to the `tab_strip` member in the future. | ||
|
||
A possible extension to the home tab is a `scope` field. This could allow for the home tab to navigate within that scope, and only open links in new tabs that are outside the scope. | ||
|
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.
It would be nice if browsers could have a keyboard shortcut to navigate to the pinned tab. Technically Ctrl-0 on chromium goes to the first tab, and maybe this would work fine.
(not a comment, just a thought)
No description provided.