-
Notifications
You must be signed in to change notification settings - Fork 56
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
"Add New Block" indicator isn't shown when title field is focused #632
Comments
This was closed automatically by the PR #1219, but I'm reopening it because there is still a remaining issue to be resolved. When an empty post is first opened, until the user interacts with the "Start writing..." portion of the post in some way (i.e., tapped on it), there will be no "Add Block Here" indicator shown when the user opens the add block bottom sheet—this includes if the user has only interacted with the post title. The technical reason for this is that the ABH indicator is attached to our block holders, but until the user taps on the "Start writing..." placeholder text in a new post, the post does not actually contain any blocks (it just has a RichText field that is replaced with a block onTouch), so there is nothing to show an ABH indicator. This can be fixed (possibly without too much difficulty), but before going down that path I wanted to see how we wanted to approach this. On one hand, not having the indicator when a new post is first created does not seem like a big deal since the entire post content ("Start writing...") will be replaced with whatever block is added. On the other hand, this is inconsistent with how this is handled in other cases. For example, if the user has a post with only a single empty paragraph block (which is what it looks like when you open a new post, but the empty paragraph block is only added once you actually interact with the post content) then the ABH indicator will appear. To be more precise, now that the initial fix for this issue is merged we have this behavior:
My take is that we probably do want the ABH indicator to show at ⬆️ Step 4 ⬆️ for the sake of consistency, but that this should not be considered a blocker for the beta release. @iamthomasbishop , what do you think? |
Fwiw, I see what you're saying, but I still think the ideal is to be consistent in terms of behavior and always show the indicator. Thanks for the detailed information on the flow! |
👍 Sounds good. I'll work on getting the ABH showing up in this last scenario (empty post where the post content has not been interacted with) and I think that should close out this issue. |
@hypest Any thoughts on whether the remaining edge case needs to be handled for the open beta? Or should the other issues be considered higher priority? My inclination would be to keep this card open so we are sure to address this edge case but to move it from the open beta column. |
It'd be best for consistency to show the ABH bar. That said, since the post is empty, there can't be too much confusion about where the new block will be added so, this now feels less of a priority, compared to other Open Beta issues. So, let's keep this open and we'll tackle it later, cool? |
Not sure if you're actively working on this one @mchowning , if so, better to move the ticket to the "In Progress" column in the project board, otherwise let's un-assign the ticket. Thanks! |
Expected behavior
If I tap the Inserter toggle to choose a new block while focused on the title, I expect the "add new block" indicator to show, just like when in the body of the post.
Actual behavior
Indicator isn't shown while focused on title.
Example:
Steps to reproduce (iOS and Android):
Potentially related issue: #543
Outstanding Tasks
The text was updated successfully, but these errors were encountered: