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

[TEST] Gutenberg - Slash inserter #16064

Closed
wants to merge 1 commit into from

Conversation

geriux
Copy link
Contributor

@geriux geriux commented Mar 11, 2021

To generate testing builds

Gutenberg Mobile PR -> wordpress-mobile/gutenberg-mobile#3250
Gutenberg PR -> WordPress/gutenberg#29772
WordsPress Android PR -> wordpress-mobile/WordPress-Android#14245

Fixes #

To test:

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.

@geriux geriux added [Status] DO NOT MERGE Gutenberg Editing and display of Gutenberg blocks. labels Mar 11, 2021
@peril-wordpress-mobile
Copy link

peril-wordpress-mobile bot commented Mar 11, 2021

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

@peril-wordpress-mobile
Copy link

peril-wordpress-mobile bot commented Mar 11, 2021

Warnings
⚠️ This PR is tagged with 'DO NOT MERGE'.

Generated by 🚫 dangerJS

@peril-wordpress-mobile
Copy link

peril-wordpress-mobile bot commented Mar 11, 2021

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

@iamthomasbishop
Copy link

iamthomasbishop commented Apr 16, 2021

EDIT: I just realized I accidentally posted this to the WPiOS PR, not the main Gutenberg one — too many open tabs! 🤦 😆 . I can re-post over there if you'd like.

@geriux @mkevins I gave the new build a spin on both platforms and it's looking and feeling great! The color refinements look solid. thanks for taking care of that 👍 I noticed one blocker on Android that I'd like to fix (if possible) before we ship this, and then I have two separate proposals which we discussed in Slack on Wed/Thu, but aren't considered blockers if we are anxious to ship this.

Issue: Blocking toolbar on Android

I just noticed on Android that the toolbar can sometimes cover the selected block if the selected block is roughly vertically centered in the viewport (video). Is there any way we can adjust the Y-position of the block/canvas a bit more to ensure that the input remains visible? Note: this is on Android — iOS seems to add an additional bit of margin so I'm not seeing the same issue, as you can see in this video)


Proposals

Again, these are not considered blockers preventing us from shipping what we have here, but I wanted to propose a couple of small refinements that we could make moving forward. We can split into separate issues if necessary.

Proposal: new italics icon

I have often felt the "italics" icon that we are using (from the WP icon set aka "G2") was a little confusing, but it hasn't caused any real conflicts until now, because the icon tends to look like a "slash", which is obviously a potential issue considering the "/" character of the placeholder/writing prompt is close enough in proximity to the "italic" icon in the block toolbar.

I found at least one issue upon a quick GitHub search, and think it'd be worth proposing some updates to the icon in core, but in the meantime I propose that we use a custom icon on mobile. A simple extension of the current italics icon would probably be clearer — something along the lines of this. If that's a route we want to go down, I can provide an SVG to add to our small set of custom icons and then get some additional design eyeballs on it for review.

Proposal: refined writing prompt

Now that I'm seeing the prompt in context, I realized we have an opportunity here (and space) to also hint at the standard (+) inserter button. This should assist in discovery (which is relevant to one of our findings from recent usability tests, wherein users needed a little nudge towards the button to discover it).

Here are 2 options — the first very straight forward and the second a little less straight forward but arguably more impactful:

  1. Change text string to: Tap (+) or type / to add blocks
  2. Change text string to the previous option, but also swap (+) for an inline icon that looks like the inserter button except desaturated and a little bit smaller

Let me know if you have any questions or concerns. I think once we're able to sort out the Android scroll position issue, let's go ahead and ship this unless you feel the proposals are worth rolling into the initial release. Thanks again for all of y'all's great work on this!

@geriux geriux force-pushed the gutenberg/try-slash-inserter branch from 01ce267 to 7d60247 Compare May 12, 2021 07:50
@geriux geriux closed this May 20, 2021
@geriux geriux deleted the gutenberg/try-slash-inserter branch May 20, 2021 09:05
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. [Status] DO NOT MERGE
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants