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

[Dev] Images not loading #2845

Closed
1 task
thetif opened this issue Feb 4, 2021 · 5 comments · Fixed by #2858
Closed
1 task

[Dev] Images not loading #2845

thetif opened this issue Feb 4, 2021 · 5 comments · Fixed by #2858
Assignees
Labels
bug needs triage, then squashing Development Issues for the dev team resolve medium Quality Quality Assurance and Quality Control sev1 Experience cannot be completed or is significantly compromised.

Comments

@thetif
Copy link
Contributor

thetif commented Feb 4, 2021

Images are not displaying in the RTE because the cookies are being blocked on staging and production

This task is done when…

This could include:

  • images are displaying
@thetif thetif added Development Issues for the dev team resolve medium labels Feb 4, 2021
@thetif thetif self-assigned this Feb 4, 2021
@thetif thetif added bug needs triage, then squashing sev1 Experience cannot be completed or is significantly compromised. labels Feb 4, 2021
@nicholeweems nicholeweems added this to the 02/08/2021 - 02/19/2021 milestone Feb 4, 2021
@thetif thetif reopened this Feb 9, 2021
@thetif thetif mentioned this issue Feb 9, 2021
9 tasks
@thetif
Copy link
Contributor Author

thetif commented Feb 16, 2021

@eStar386 can you verify that this is working in Staging too (you can use the same users that are in keybase) once you verified it you can move to to ready to review and tag Jerome and Jenn. Thanks!

@eStar386
Copy link
Contributor

@thetif Yes to both and NP!

@eStar386
Copy link
Contributor

eStar386 commented Feb 19, 2021

@thetif , I added 2605 because this accessibility ticket directly impacts the image functionality. Any Dependencies I add shouldn't be taken as needed to be completed, I am just adding them because i need to test that it works for those specified components and this function will need to be retested @jeromeleecms thoughts?

@eStar386
Copy link
Contributor

Failed Accessibility Testing - There are two small accessibility issues in the RTE.

  1. Wave Error - Steps to Reproduce Right click and select Wave. It shows a broken ARIA menu.
  2. Screenreader- Steps to Reproduce. Using voiceover, have the screenreader read the menu for the rich text editor. It reads all the menu items except for the paragraph - it goes div div div paragraph.

Functional Issue: There is a bug with the cookie being rejected or blocked. In these cases, it causes a 401 error and displays the following the unhelpful - the user needs to know to enable cookies to fully use the site. Otherwise, they can't login and won't know why.
image
Browser:Google
Steps To Reproduce:

  1. Disable or Block the Eapd Cookie
  2. Login to the apd.
    Result: The image above.

@eStar386 eStar386 added the Quality Quality Assurance and Quality Control label Mar 2, 2021
@eStar386
Copy link
Contributor

eStar386 commented Mar 15, 2021

Documenting a known accessibility issue with the TinyMCE headers causing the above wave errors. Also, documented in #2675.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug needs triage, then squashing Development Issues for the dev team resolve medium Quality Quality Assurance and Quality Control sev1 Experience cannot be completed or is significantly compromised.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

5 participants