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
[DDW-486] Close Newsfeed on Blur #2250
Conversation
Since we are currently not optimizing for screen readers
Hi @DominikGuzei . For build 15340: All looks good to me except when I clicked on the grey out area of "Join" and "Import", the Newsfeed didn't close. Please see recording here |
Hi @DominikGuzei. Very minor or perhaps not an issue at all. Similar to Miors findings when the user clicks on Language, Number Format, Date Format or time format the Newsfeed does not automatically close. See video |
Thanks, @mioriohk, and @ManusMcCole - I will look into these issues 👍 |
@mioriohk @ManusMcCole please re-check, I generalized the logic so it should work in any scenario now! |
@DominikGuzei , this is the current findings for build 15388. Only pending issue listed. I have moved fixed issue into testing summary above. Pending |
@mioriohk I also tested this and it worked for me 🤔 |
@mioriohk oh I just double-checked and indeed it doesn't work 👍 will debug |
@DominikGuzei is this ready for testing again? |
@nikolaglumac @mioriohk yes, ready for final testing 👍 |
Hi @DominikGuzei . Looks good to me on build 15390 Fixed |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Great job @DominikGuzei . Approved. 👍
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good 👍
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Great work @DominikGuzei 💯
This PR implements the small UX feature of closing the Newsfeed when clicking anywhere else in the UI:
Screenshots
Testing Checklist
Testing Summary
For build 15390
Fixed
1 - When the user clicks on Language, Number Format, Date Format or time format the Newsfeed does not automatically close. See video
2 - When I clicked on the grey out area of "Join" and "Import", the Newsfeed didn't close. Please see recording here. Addition: I have taken a screenshot and highlighted the problematic areas in red rectangles See here
Review Checklist
Basics
feature
/bug
/chore
,release-x.x.x
)yarn test
)yarn dev
)yarn package
/ CI builds)yarn flow:test
)yarn lint
)yarn prettier:check
)yarn manage:translations
produces no changes)yarn storybook
)yarn.lock
file is updatedCode Quality
Testing
After Review
done
column on the YouTrack board