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

Portfolio Senior Official page #2378

Closed
5 tasks done
abroddrick opened this issue Jun 25, 2024 · 2 comments · Fixed by #2562
Closed
5 tasks done

Portfolio Senior Official page #2378

abroddrick opened this issue Jun 25, 2024 · 2 comments · Fixed by #2562

Comments

@abroddrick
Copy link
Contributor

abroddrick commented Jun 25, 2024

Description

Both in the org mode and for non-org members we need the senior official table to be non-editable but displayed more like text instead of disabled fields. This helps with accessibility, especially keyboard navigability. To reduce context switching and in hopes of re-using the same template between both the org model and non-org model pages, this ticket will apply to both

Acceptance Criteria

  • In the portfolio view the senior official side navigation is a clickable link to the senior official page
  • both in the portfolio page matches the figma in the previous AC
  • both pages should be non-editable
  • the page pulls from the portfolio Senior official (not the SO on a particular domain)
  • Non org member page still uses the the SO that is from the domain information

Additional Context

No response

Issue Links

@abroddrick abroddrick added the story User stories label Jun 25, 2024
@abroddrick abroddrick added dev and removed story User stories labels Jun 25, 2024
@abroddrick
Copy link
Contributor Author

abroddrick commented Jul 1, 2024

@katypies did you add accessibility by access? usually that is for 508 accessibility stuff
Nevermind I see why you did it

@abroddrick
Copy link
Contributor Author

Tentatively slotting for sprint 50 as the blocking ticket is in sprint 49, so if that one is done and all other tickets are finished early this would be a good one to be pulled in. CC: @vickyszuchin, I doubt devs will finish all the tickets while I am gone, but CCing you just in case

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment