Commenting does not auto scroll user
Description
Upon clicking on the comment button, the page should autofocus and a scroll user to the right height.
Steps to reproduce
- Create an external share with commenting privilege
- As an external user click on the comment button
- Notice that the page is not scrolling to the appropriate height
Actual result
User must scroll manually.
Expected result
User is automatically focused on the appropriate height for commenting
Activity
Show:
Details
Priority
MediumAssignee
Jakub Myśliński
Reporter
Parsa Shiva
Labels
ui/ux
Fix versions
Roadmap
Components
shared_page
Created
26 June 2023, 10:21
Updated
31 August 2023, 16:54
More fields
Created: 26 June 2023, 08:21
Updated:
31 August 2023, 14:54
Hello @Jakub Myśliński,
Please merge code to dev branch.
This is the best moment to add more information that can be helpful to prepare release notes.
Can you prepare short overview of change that can be used in release notes?
Please provide short GIF that showcase feature.
If GIF make no sense, can you provide image that highlights feature that can be used in release notes (cropped & annotated)?
@Jakub Myśliński Fix verified - QA environment.
Hello @Jakub Myśliński ]
This is the best moment to add more information that can be helpful for tester.
What areas are affected?
What are potential edge cases?
Was it checked for XSS problems?
Does change affect security, is new data exposed?
Please attach - Before / After screenshot if possible.
Hello @Miłosz Jaskólski,
Task is ready for review.
@Jakub Myśliński please make sure reviewer
have easy access to contend to be reviewed.
If this is code change. Please make sure PR is created.
If this is new documentation, blogpost, etc. Please provide link to page.
@Jakub Myśliński Following issue occurs - QA environment.
The auto scroll is not displaying the whole field, it needs to be realigned.
Hello @Jakub Myśliński ]
This is the best moment to add more information that can be helpful for tester.
What areas are affected?
What are potential edge cases?
Was it checked for XSS problems?
Does change affect security, is new data exposed?
Please attach - Before / After screenshot if possible.
Hello @Jakub Myśliński,
Change was reviewed and approved.
Task is ready to be deployed to QA.
Once it is deployed to QA please move ticket to "To Test"
Thanks you!
Hello @Krzysztof Bogdan,
Task is ready for review.
@Jakub Myśliński please make sure reviewer
have easy access to contend to be reviewed.
If this is code change. Please make sure PR is created.
If this is new documentation, blogpost, etc. Please provide link to page.
Hello @Krzysztof Bogdan
This is the best moment to add more description, wire-frames, comments.
Why we doing this?
What is expected result?