robby1066 / keepposted-help

Documentation, Issue Tracker, and Repository of Knowledge For Keep Posted (www.keepposted.io) — A way to keep teams informed and connected without disrupting their focus.
https://www.keepposted.io
1 stars 0 forks source link

The 'detailed feedback' on message display on mobile is missing and a green block is displayed instead #51

Closed robby1066 closed 3 years ago

robby1066 commented 3 years ago

Describe the bug

At the end of a message, there is supposed to be an option for structured feedback, and detailed, free text feedback.

On mobile, only the structured feedback is shown and beneath it is a green block where the detailed feedback section should be.

Steps To Reproduce

  1. Go to a message on mobile. example
  2. Watch the video (or skip to the end until the replay or feedback options appear)
  3. Scroll past the structured feedback section
  4. Observe that the detailed feedback section is missing and there is a green block in it's place.

Expected behavior

The viewer should be able to access the detailed feedback block

Screenshots

Screenshot 2021-03-05 at 16 29 40

robby1066 commented 3 years ago

This may be related to #30 and how the detailed feedback section is hidden from the tab order.

robby1066 commented 3 years ago

This is fixed and ready for deploy. The change effectively removes the special visibility: hidden tweaks to manage the focus order, but only on mobile. I'm unsure of the accessibility fallout of that. Since the elements that were getting hidden should actually be visible at all times in the mobile view, this shouldn't have a negative impact on the focus order. But there may be some use cases I'm not thinking about.

robby1066 commented 3 years ago

In testing, I realized this fix somehow hides the 'go faster' button on mobile. Not sure why that is. But this issue can't be closed until that's fixed.

robby1066 commented 3 years ago

Found the problem. The accessibility visibility rules in the CSS file were right next to the rule that managed the visibility of the "go faster" button. I overzealously copy / pasted when I moved the accessibility stuff to the desktop only section 🤦‍♂️.

Fixed and will go out in the next deploy.

robby1066 commented 3 years ago

All bugs are fixed and tested. Closing this.