-
-
Notifications
You must be signed in to change notification settings - Fork 10.6k
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
Zelda Audit #5652
Comments
refs TryGhost#5652 - stops spinner from spinning forever on settings screen when saving
@JohnONolan the reason the width of the publish button is being set with JS is because when the spinner is activated on post save, the width of the button will shrink to the size of the spinner. This can be removed; however, I didn't have any other good ideas on what to do to prevent the button from shrinking when the spinner is active. |
@acburdine The way we handle this on Ghost.org is that when a loading-button is clicked:
(so width is only fixed when it needs to be, not all the time) |
@JohnONolan ah gotcha. That's kind of the way I was doing it, but not exactly, so I'll PR a fix to it. |
refs TryGhost#5652 - changes the spin-button so that it only sets the size when the button is submitting instead of all of the time
I've just seen the same issue in Safari too |
issue TryGhost#5652 - display "Sending Invite..." text after clicking "RESEND" link - change "Invitation sent" alert to notification
issue TryGhost#5483 & TryGhost#5652 - trigger the close menus action every time a route transition is successful - close nav menu when clicking "collapse sidebar" button (action chain was incomplete)
issue TryGhost#5652 - highlight input border and response text in red
refs TryGhost#5652 - new action on focusOut which only calls validate if the fields has a value
refs TryGhost#5343, TryGhost#5652 - implements basic post and user search using selectize input - queries minimal API endpoint and refreshes results on search input focus if results are older than 60 seconds
ref TryGhost#5652 - validations can be in default, success or error state - adds check for 'hasValidated' if the validations haven't fired yet, the field is in the default state - hasValidated is an Ember.Array which tracks the state for each field
refs TryGhost#5652 - on mobile, add Markdown/Preview links to the editor footer that switch between the respective views - temporary resolution to the inability to upload on mobile without access to the preview
refs TryGhost#5652 - handles the skip link using js so that ember doesn't interfere with the anchor - uses the code suggested here: https://www.codehive.io/boards/pZUuwIk
refs TryGhost#5652 - reverts previous fix (TryGhost#5698) to return translate3d smoothness - add separate open-autonav toggle element and style it to always stick to the viewport edge to avoid wandering hover hit areas
refs TryGhost#5652 - with these changes, validation appears, but doesn't properly prevent closing the modal - this needs revisiting at some point
Items are all checked off! |
Followup of #5314
ALL TESTED IN SAFARI 8.0.7 for OSX - I am actively working on these issues
By Hannah
gravatar behaviour doesn't fire until blurmoved to Revisit gravatar aspect of image component on setup/signup #5797loading interstitial & pre-app loading screenmoving to post-zeldaLoading buttons: Don't know how to solve
When an action takes more than 1s, the loading spinner makes sense in a button. But when an action takes 0.1s - adding/removing a loading spinner makes the button feel broken. I think we need a mandatory "success" animation which can follow the loading animation when the route is not changing. I don't think this is going to be easy.
Also: When logging in, the loading button stops spinning before the admin loads. This should not happen. It should keep spinning all the way until the route changes without ever stopping
Mobile
All tested on latest Chrome for iOS
The text was updated successfully, but these errors were encountered: