Skip to content

[Gutenberg] Implementing Ghost loading view for Gutenberg#14310

Merged
etoledom merged 13 commits intodevelopfrom
gutenberg/improve-editor-loading-experience
Jul 1, 2020
Merged

[Gutenberg] Implementing Ghost loading view for Gutenberg#14310
etoledom merged 13 commits intodevelopfrom
gutenberg/improve-editor-loading-experience

Conversation

@etoledom
Copy link
Copy Markdown
Contributor

@etoledom etoledom commented Jun 12, 2020

Fixes wordpress-mobile/gutenberg-mobile#1179

Gutenberg-mobile side PR: wordpress-mobile/gutenberg-mobile#2382

This PR implements a loading view for Gutenberg using the Ghost Views used in the WPiOS App.

To test:

  • Run the app.
  • Open a post or page with Gutenberg.
  • Check that the loading view shows similar to the gif attached.
  • Check light and dark mode, portrait and landscape.

skeleton

PR submission checklist:

  • I have considered adding unit tests where possible.
  • I have considered adding accessibility improvements for my changes.
  • I have considered if this change warrants user-facing release notes and have added them to RELEASE-NOTES.txt if necessary.

@peril-wordpress-mobile
Copy link
Copy Markdown

peril-wordpress-mobile bot commented Jun 12, 2020

You can trigger an installable build for these changes by visiting CircleCI here.

@etoledom etoledom self-assigned this Jun 12, 2020
@etoledom etoledom added the Gutenberg Editing and display of Gutenberg blocks. label Jun 12, 2020
@etoledom etoledom added this to the 15.2 milestone Jun 12, 2020
@etoledom etoledom marked this pull request as ready for review June 12, 2020 18:30
Copy link
Copy Markdown
Contributor

@guarani guarani left a comment

Choose a reason for hiding this comment

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

Looks great to me! Tested on dark/light mode on iPhone and iPad.

@iamthomasbishop
Copy link
Copy Markdown

This is looking so good @etoledom! I'd like to try out a test build before giving this a final sign-off, but it's looking really solid — title/body items seem to line up really well and the timing feels pretty good.

I have one small request, which I think only applies to dark mode: Can we fill the background of the "ghost" block toolbar in the same color as the "real" block toolbar? The items over the toolbar would then need to use a slightly lighter gray. I think this probably isn't an issue in light mode bc the toolbar is the same color as the background.

@etoledom
Copy link
Copy Markdown
Contributor Author

Thank you both for your reviews!

I have implemented the requested improvements from @iamthomasbishop :

Dark Mode Light Mode
ghost-dark ghost-light

I have also triggered the Test Build to be able to see it better than with gifs 👍

@peril-wordpress-mobile
Copy link
Copy Markdown

peril-wordpress-mobile bot commented Jun 16, 2020

You can trigger optional UI/connected tests for these changes by visiting CircleCI here.

@iamthomasbishop
Copy link
Copy Markdown

@etoledom this is feeling super smooth. Funny enough, the editor loads so quickly in most cases that you don't see the loading state for more than a split second — obviously a great problem to have 😄

I noticed 1 tiny thing — in dark mode, can we remove the border-top on the block toolbar while loading?

@pablohoneyhoney
Copy link
Copy Markdown

Does it need to be different than browser's? And why?
If it does, would it make sense to align there?

@iamthomasbishop
Copy link
Copy Markdown

@pablohoneyhoney AFAIK this is using our fully-native loader components (which use the system-defined colors). It’s feels pretty similar to what exists on web, so I think we are in good shape WRT alignment.

@etoledom
Copy link
Copy Markdown
Contributor Author

in dark mode, can we remove the border-top on the block toolbar while loading?

Done!

AFAIK this is using our fully-native loader components

That's correct, at this point everything is pure native.

@iamthomasbishop - ready for another look 👀

@etoledom etoledom added Gutenberg Editing and display of Gutenberg blocks. and removed Gutenberg Editing and display of Gutenberg blocks. labels Jun 18, 2020
@etoledom etoledom requested a review from guarani June 18, 2020 11:16
@etoledom
Copy link
Copy Markdown
Contributor Author

@guarani - There has been quite a few changes since the last review. Would you mind taking a fast look again?
Thanks!

@guarani
Copy link
Copy Markdown
Contributor

guarani commented Jun 18, 2020

Tested and working great! LGTM @etoledom!

@pablohoneyhoney
Copy link
Copy Markdown

@pablohoneyhoney AFAIK this is using our fully-native loader components (which use the system-defined colors). It’s feels pretty similar to what exists on web, so I think we are in good shape WRT alignment.

If we consider the below, there are ways to have more proximity @iamthomasbishop

Screen Shot 2020-06-18 at 1 11 35 PM

@iamthomasbishop
Copy link
Copy Markdown

Ah, fair enough @pablohoneyhoney. I was thinking the ghost items on the web were also positioned over the actual UI elements (as we are doing on mobile, where the ghost elements are sized and positioned over the title, body, and block toolbar). Do you think it'd make sense for web to adopt the same concept? If so, it shouldn't be a huge challenge, although the web UI changes so often that we'd want to keep an eye on it over time.

@etoledom
Copy link
Copy Markdown
Contributor Author

If we consider the below, there are ways to have more proximity @iamthomasbishop

I agree with @iamthomasbishop

The web ghost resembles the top toolbar and content. On mobile we don't have top toolbar so it makes sense to resemble the mobile layout with the toolbar at the bottom.

@etoledom etoledom force-pushed the gutenberg/improve-editor-loading-experience branch from 3147d7c to 1e7cb4b Compare June 26, 2020 09:38
@etoledom etoledom changed the base branch from develop to gutenberg/after_1.31.0 June 26, 2020 09:39
@etoledom
Copy link
Copy Markdown
Contributor Author

Updated to target gutenberg/after_1.31.0

@iamthomasbishop - Do we have your ✅ ?

@SergioEstevao
Copy link
Copy Markdown
Contributor

SergioEstevao commented Jun 26, 2020

@etoledom can you please recreate this PR and target the develop branch. It got closed automatically when I deleted the after branch. Sorry about that.

I reopened the after branch if you want you can merge there if you want to target 15.3 release.

If you want to target 15.2 you can target directly develop.

@SergioEstevao SergioEstevao reopened this Jun 26, 2020
@etoledom etoledom changed the base branch from gutenberg/after_1.31.0 to develop June 29, 2020 14:20
@jkmassel
Copy link
Copy Markdown
Contributor

Howdy friends – we're freezing 15.2 today, so I'm bumping this PR forward to 15.3. If it needs to land in the 15.2 series, please merge it to release/15.2 and DM me – I'll be happy to create a new beta release!

@jkmassel jkmassel modified the milestones: 15.2, 15.3 Jun 29, 2020
@iamthomasbishop
Copy link
Copy Markdown

@etoledom just tested on iPhone and iPad, and it’s really feeling great! I think it’s ready to ship! Now we just need to implement this on Android ;)

@etoledom etoledom merged commit 3e93a60 into develop Jul 1, 2020
@etoledom etoledom deleted the gutenberg/improve-editor-loading-experience branch July 1, 2020 09:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Gutenberg Editing and display of Gutenberg blocks.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Try] Improve editor loading experience

6 participants