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

Four Kitchens Release - Sprint 39 #1400

Merged
merged 11 commits into from
Nov 27, 2023
Merged

Four Kitchens Release - Sprint 39 #1400

merged 11 commits into from
Nov 27, 2023

Conversation

cienvaras
Copy link
Collaborator

@cienvaras cienvaras commented Nov 20, 2023

READY FOR REVIEW

Summary

  • SHS-5130: Implementation: Add <hr> tag to Color Band if no text
  • SHS-5113: Editorially Customizations
  • SHS-5119: Bug: Gray overlay (z-index issue?) on Manage Layout when clicking Configure for a field or block
  • SHS-5095: Need better UI/theme around login page
  • SHS-5148: Bug: hb-categories link hover style generally missing underline on last line
  • SHS-5086: Add the Delete option back for Developers
  • SHS-5133: Remove redundant aria-hidden_tabindex and add alt on images in links

Need Review By (Date)

11/29

Urgency

medium

Steps to Test

SHS-5130

  1. Edit a page
  2. Add a new page component, and choose Color Band
  3. Do not add text to the text field and save
  4. Ensure that if the color band has no text, an <hr> tag prints inside the color band.
  5. Ensure that color bands with text still print the same as before.

SHS-5113

  1. Visit any page as a "Developer" user. You should see the Editoria11y widget in the bottom right corner
  2. Click on the widget. You should see a panel with three tabs. The second tab should have the title "Heading Outline" instead of "Outline"
    Screenshot 2023-11-23 at 13 21 41
  3. Visit a page with an Editoria11y alert status (red outline widget). If you can't find one, you can create a flexible page and add an empty heading in the Text Area. Confirm that the widget error count text is black instead of red.
    Screenshot 2023-11-23 at 13 25 19

SHS-5119

1.Visit a Layout Builder Page like https://hs-colorful.suhumsci.loc/admin/structure/types/manage/hs_person/display/default/layout
2. Click on a Pencil icon next to one of the blocks, and then click "Configure"
3. Ensure that the modal window appears in front of the grayed-out panel (instead of behind it.)

SHS-5095

  1. Visit the user login page, like https://hs-colorful.suhumsci.loc/user/login
  2. Ensure that the page is using the custom theme, and not Gin Admin theme.
  3. Ensure the there is only one title, the box-shadows have been removed, and the title is black (and not a light gray.)

SHS-5148

  1. Visit https://hs-colorful.suhumsci.loc/default-views/people
  2. Confirm that the "Research Areas" items present in some cards have a bottom border on hover
  3. Repeat test on https://hs-traditional.suhumsci.loc/default-views/people
Screenshot 2023-11-27 at 10 28 40

SHS-5086

  1. Visit nodes of different content types as an user with the "Developer" role. You should be able to see and use the "Delete" local task
  2. Repeat the test with a couple of other users with different roles. The "Delete" local task shouldn't be present

SHS-5133: Remove redundant aria-hidden_tabindex and add alt on images in links

  1. Visit a page with cards that have a linked image in them, like https://english.suhumsci.loc/
  2. Ensure that on cards that have linked images, the images no longer have the aria-hidden: true and tab-index: -1 attributes.

PR Checklist

@cienvaras cienvaras self-assigned this Nov 20, 2023
mndonx and others added 5 commits November 20, 2023 16:42
* feat(SHS-5095): theming the user login page

* fix(SHS-5095): adding composer lines back in, exporting config

* fix(SHS-5095): adding composer lines back in
* fix(shs-5130): adding hr tag when there is no color band text

* fix(SHS-5130): hr adjustments

* fix(SHS-5130): hr margins and another instance
Base automatically changed from 10.0.3-release to develop November 27, 2023 17:56
@cienvaras cienvaras changed the base branch from develop to 10.0.4-release November 27, 2023 18:04
Base automatically changed from 10.0.4-release to develop November 27, 2023 21:49
cienvaras and others added 4 commits November 27, 2023 17:01
* feat(shs-5113): editoria11y customizations

* feat(shs-5113): Editoria11y customizations

* fix(shs-5113): update color value in editoria11y styles

Co-authored-by: Amanda Luker <[email protected]>

* chore(shs-5113): update compiled theme js

---------

Co-authored-by: Amanda Luker <[email protected]>
@cienvaras cienvaras marked this pull request as ready for review November 27, 2023 23:41
@cienvaras cienvaras requested review from pookmish and joegl November 27, 2023 23:41
@cienvaras cienvaras assigned joegl and pookmish and unassigned cienvaras Nov 27, 2023
@pookmish pookmish changed the base branch from develop to 10.0.5-release November 27, 2023 23:45
@pookmish pookmish merged commit ce6f28a into 10.0.5-release Nov 27, 2023
1 check passed
@pookmish pookmish deleted the fk-stnfd-sprint-39 branch November 27, 2023 23:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants