Skip to content
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

Merged
merged 6 commits into from
May 27, 2022
Merged

Create tabbed-mode-explainer.md #55

merged 6 commits into from
May 27, 2022

Conversation

loubrett
Copy link
Collaborator

No description provided.

Copy link
Contributor

@tomayac tomayac left a 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.

tabbed-mode-explainer.md Outdated Show resolved Hide resolved
tabbed-mode-explainer.md Show resolved Hide resolved
tabbed-mode-explainer.md Outdated Show resolved Hide resolved
tabbed-mode-explainer.md Outdated Show resolved Hide resolved
tabbed-mode-explainer.md Outdated Show resolved Hide resolved
loubrett and others added 2 commits May 18, 2022 06:33
Co-authored-by: Thomas Steiner <tomac@google.com>
Copy link
Collaborator

@alancutter alancutter left a 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!

tabbed-mode-explainer.md Outdated Show resolved Hide resolved
tabbed-mode-explainer.md Outdated Show resolved Hide resolved
tabbed-mode-explainer.md Outdated Show resolved Hide resolved
tabbed-mode-explainer.md Outdated Show resolved Hide resolved
tabbed-mode-explainer.md Outdated Show resolved Hide resolved
tabbed-mode-explainer.md Outdated Show resolved Hide resolved
loubrett and others added 2 commits May 19, 2022 02:55
Co-authored-by: Thomas Steiner <tomac@google.com>
Copy link
Collaborator

@dmurph dmurph left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM with nit

tabbed-mode-explainer.md Outdated Show resolved Hide resolved
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.

Copy link
Collaborator

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)

@loubrett loubrett merged commit b0562de into gh-pages May 27, 2022
@loubrett loubrett deleted the loubrett-patch-1 branch May 27, 2022 04:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
4 participants