Skip to content
This repository was archived by the owner on Nov 22, 2024. It is now read-only.

View Source should always open a ChromeCustomTab, fix #41#42

Open
SimonMarquis wants to merge 1 commit into
JakeWharton:masterfrom
SimonMarquis:patch/view_source_custom_tabs
Open

View Source should always open a ChromeCustomTab, fix #41#42
SimonMarquis wants to merge 1 commit into
JakeWharton:masterfrom
SimonMarquis:patch/view_source_custom_tabs

Conversation

@SimonMarquis

Copy link
Copy Markdown

Added a helper class with companion methods to build the root CustomTabsIntent and share the same default resources: color, default action, maybe animations?

@SimonMarquis SimonMarquis changed the title Fix #41, View Source should always open a ChromeCustomTab View Source should always open a ChromeCustomTab, fix #41 Jan 17, 2018
@SimonMarquis SimonMarquis changed the title View Source should always open a ChromeCustomTab, fix #41 View Source should always open a ChromeCustomTab, fixes #41 Jan 17, 2018
@SimonMarquis SimonMarquis changed the title View Source should always open a ChromeCustomTab, fixes #41 View Source should always open a ChromeCustomTab, fix #41 Jan 17, 2018
@jaredsburrows

Copy link
Copy Markdown

Can you post screenshots?

@SimonMarquis

Copy link
Copy Markdown
Author

What kind of screenshot do you want since there is no UI changes 😕 ?
The View source was already opened in a CustomTab if it was launched from the main screen.
I can make a before/after gif of the changes if you want.

Added a helper class with companion methods to build the root CustomTabsIntent and share the same default resources: color, default action, maybe animations?
@SimonMarquis

Copy link
Copy Markdown
Author
Before After
before after

@jaredsburrows

Copy link
Copy Markdown

@SimonMarquis This looks great!
@JakeWharton

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants