Skip to content

set first column active on load#169

Merged
Gaurav0 merged 1 commit intoember-cli:masterfrom
kagemusha:master
Aug 24, 2015
Merged

set first column active on load#169
Gaurav0 merged 1 commit intoember-cli:masterfrom
kagemusha:master

Conversation

@kagemusha
Copy link
Copy Markdown
Contributor

Small change, but currently when page loads and code panes are greyed out, can give impression that twiddle is disabled. This seeks to highlights that is editable and people can jump right in.

Comment thread app/gist/controller.js Outdated
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.

Column#active is a computed property with no setter.

The correct way to make a column active is to set activeEditorCol on the controller.

@Gaurav0
Copy link
Copy Markdown
Contributor

Gaurav0 commented Aug 23, 2015

This PR causes the first column to always be active.

@kagemusha
Copy link
Copy Markdown
Contributor Author

@Gaurav0 thanks for the insight. edited based on your suggestion.

Gaurav0 added a commit that referenced this pull request Aug 24, 2015
set first column active on load
@Gaurav0 Gaurav0 merged commit 1621ae5 into ember-cli:master Aug 24, 2015
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