Skip to content

better sign in that fixes #183 & hack that fixes #187 ....again#190

Merged
Gaurav0 merged 3 commits intoember-cli:masterfrom
KTKate:betterSignIn
Sep 24, 2015
Merged

better sign in that fixes #183 & hack that fixes #187 ....again#190
Gaurav0 merged 3 commits intoember-cli:masterfrom
KTKate:betterSignIn

Conversation

@KTKate
Copy link
Copy Markdown
Contributor

@KTKate KTKate commented Sep 7, 2015

Current

twiddle-signin-old

### This PR

twiddle-signin-new

I moved the github icon to the right because it looked disjointed on the left somehow. Once signed in, the user's avatar is unchanged.

Comment thread app/components/user-menu.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.

computed.readOnly

@stefanpenner
Copy link
Copy Markdown
Contributor

looks great

@KTKate
Copy link
Copy Markdown
Contributor Author

KTKate commented Sep 7, 2015

Haha as soon as I submitted the PR I realized that would have been better. I'll change it.

Comment thread app/gist/new/route.js
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.

Adding a space here won't fix it if the user changes the name.

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.

Hm, I wasn't aware the problem happened when the title had been changed. Even if the named has been changed and then later reverted back to New Twiddle. Please show steps to reproduce a once edited title getting cut off.

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.

So you have to

  1. Change the title
  2. Save the gist
  3. Load Ember Twiddle with the saved gist.

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.

I tried that before and just tried it again and the title was not cut off. I tried it both logged in and logged out. This issue is unpredictable enough it's almost like a practical joke!

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.

Yes, I know. Sorry. Maybe instead of adding a space to the value you can add a space to a computed property based on value and show that?

Gaurav0 added a commit that referenced this pull request Sep 24, 2015
better sign in that fixes #183 & hack that fixes #187 ....again
@Gaurav0 Gaurav0 merged commit 93661e4 into ember-cli:master Sep 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.

3 participants