Skip to content

Offcanvas example: transition the transform - #28203

Merged
XhmikosR merged 1 commit into
v4-devfrom
v4-dev-xmr-offcanvas
Feb 8, 2019
Merged

XhmikosR merged 1 commit into
v4-devfrom
v4-dev-xmr-offcanvas

Conversation

@XhmikosR

@XhmikosR XhmikosR commented Feb 7, 2019

Copy link
Copy Markdown
Member

This is more efficient.

Opening this so that we don't forget about it.

@XhmikosR
XhmikosR requested a review from a team as a code owner February 7, 2019 18:02
@XhmikosR
XhmikosR force-pushed the v4-dev-xmr-offcanvas branch from b1322f9 to 3d3eacd Compare February 8, 2019 08:47
@XhmikosR

XhmikosR commented Feb 8, 2019

Copy link
Copy Markdown
Member Author

Not sure about removing visibility. Let's put it on hold for now.

@ysds

ysds commented Feb 8, 2019

Copy link
Copy Markdown
Contributor

Please see #25524 for why visibility is necessary.

@XhmikosR
XhmikosR force-pushed the v4-dev-xmr-offcanvas branch from 3d3eacd to 443abdd Compare February 8, 2019 10:49
@XhmikosR XhmikosR removed the on-hold label Feb 8, 2019
@XhmikosR
XhmikosR force-pushed the v4-dev-xmr-offcanvas branch from 443abdd to dfc5a16 Compare February 8, 2019 10:50
@XhmikosR

XhmikosR commented Feb 8, 2019

Copy link
Copy Markdown
Member Author

OK, I added back visibility.

@ysds

ysds commented Feb 8, 2019

Copy link
Copy Markdown
Contributor

These changes increase the file size, but it will be more efficient in the future.

@XhmikosR

XhmikosR commented Feb 8, 2019

Copy link
Copy Markdown
Member Author

We need to promote the best practices.

@ysds

ysds commented Feb 8, 2019

Copy link
Copy Markdown
Contributor

If I am not misunderstanding your comment, I agree with these changes.

@XhmikosR

XhmikosR commented Feb 8, 2019

Copy link
Copy Markdown
Member Author

Yeah, we say the same thing :)

@XhmikosR
XhmikosR merged commit c56b10c into v4-dev Feb 8, 2019
@XhmikosR
XhmikosR deleted the v4-dev-xmr-offcanvas branch February 8, 2019 16:08
@mdo mdo mentioned this pull request Feb 8, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants