Skip to content

Sidebar#687

Merged
Gaurav0 merged 22 commits intoember-cli:masterfrom
knownasilya:sidebar
Mar 1, 2020
Merged

Sidebar#687
Gaurav0 merged 22 commits intoember-cli:masterfrom
knownasilya:sidebar

Conversation

@knownasilya
Copy link
Copy Markdown
Contributor

@knownasilya knownasilya commented Feb 29, 2020

Pulled out some of the initial changes from #490 to get it in.

twiddle-sidebar
Screen Shot 2020-02-29 at 5 11 10 PM
Screen Shot 2020-02-29 at 5 11 19 PM

@knownasilya knownasilya requested a review from Gaurav0 February 29, 2020 04:50
},

didBecomeReady() {
didBecomeReadyOptional() {
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Ugh. Can't believe I wrote this. :)

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

We all have these moments :D

Comment thread app/styles/_toolbar.scss
height: $topbar-height;
}

@media (max-width: $screen-md-min) {
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Is this still responsive?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

No, that is my next task haha.

@knownasilya knownasilya requested a review from Gaurav0 February 29, 2020 22:37
@Gaurav0 Gaurav0 merged commit 5186d37 into ember-cli:master Mar 1, 2020
@knownasilya knownasilya deleted the sidebar branch March 1, 2020 22:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants