Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Post-GUI-Update Changes #3810

Closed
10 tasks done
JohnONolan opened this issue Aug 18, 2014 · 13 comments
Closed
10 tasks done

Post-GUI-Update Changes #3810

JohnONolan opened this issue Aug 18, 2014 · 13 comments
Labels
bug [triage] something behaving unexpectedly P2 - High [triage] High priority for immediate patch release
Milestone

Comments

@JohnONolan
Copy link
Member

As a part of the new GUI update - we are introducing many brand new (shiny!) bugs! This issue is intended to cover the things that need some additional TLC

Let's keep all New-GUI related bugs in the list above. Discussion below. Check em off as you go.

Fucking poetry.

@JohnONolan JohnONolan changed the title Post GUI-Update Changes Post-GUI-Update Changes Aug 18, 2014
@felixrieseberg
Copy link
Member

I'm gonna go ahead and try to convince the 404 page to center the error vertically. Also, the new GUI is pretty sweet.

felixrieseberg added a commit to felixrieseberg/Ghost that referenced this issue Aug 18, 2014
See TryGhost#3810
- Only thing missing: <div id=“container”></div>. Added.
@felixrieseberg
Copy link
Member

CSS bug, fix vertical centering on 404 page: fixed with #3812.

@novaugust
Copy link
Contributor

re: ember love for mobile transitions on settings views, #3259 will be a good reference to the code that use to do this

@ghost ghost mentioned this issue Aug 20, 2014
@ErisDS ErisDS added this to the 0.5.x Feature Release Backlog milestone Aug 20, 2014
@ErisDS
Copy link
Member

ErisDS commented Aug 21, 2014

A new thing I just spotted: signin screen on mobile on master currently looks like this (update fixed via ghost-ui):

@felixrieseberg
Copy link
Member

@ErisDS I actually can't reproduce that (iOS 7.1.2, Mobile Safari & Chrome). More direct pointers to how you got it to be borked?

@ErisDS
Copy link
Member

ErisDS commented Aug 21, 2014

@felixrieseberg sorry, the convo continued on IRC, turned out it was fixed in ghost-ui and hadn't been published.

@ErisDS
Copy link
Member

ErisDS commented Aug 23, 2014

On the users screen, the settings menu which provides the delete user and transfer ownership options is missing.

@felixrieseberg
Copy link
Member

I worked on that menu before, I'm sure I can bring it back.

@ErisDS
Copy link
Member

ErisDS commented Aug 23, 2014

@felixrieseberg there needs to be a design for it I think?

@felixrieseberg
Copy link
Member

Well, let's wait for that then ;-) (I assumed we'd stick with the cog, but there are surely better ways, given that the menu currently has at most two options).

@novaugust
Copy link
Contributor

"New Ember love to handle transitioning between settings views on mobile" is taken care of with #3862

update merged -- @JohnONolan you can check off the second box up there

novaugust added a commit to novaugust/Ghost that referenced this issue Sep 3, 2014
Closes TryGhost#1990, Ref TryGhost#3810
- Add `js-` prefix to jQuery calls throughout code and templates
- move MarkdownComponent classNames to template
- Fix back button on posts content preview
- Remove dead code for preview tabbing jQuery
@JohnONolan
Copy link
Member Author

Added to task list, Regression: Lost settings nav icons

image

@ErisDS ErisDS added the P2 - High [triage] High priority for immediate patch release label Sep 16, 2014
@PaulAdamDavis PaulAdamDavis mentioned this issue Sep 17, 2014
5 tasks
@ErisDS
Copy link
Member

ErisDS commented Sep 19, 2014

I believe this list is now complete and the issue can be closed.

@ErisDS ErisDS closed this as completed Sep 19, 2014
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug [triage] something behaving unexpectedly P2 - High [triage] High priority for immediate patch release
Projects
None yet
Development

No branches or pull requests

4 participants