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

SHS-5845: Collection with Well color for a page with a side bar menu #1639

Merged
merged 3 commits into from
Oct 9, 2024

Conversation

cienvaras
Copy link
Collaborator

@cienvaras cienvaras commented Sep 30, 2024

READY FOR REVIEW

Summary

  • Fix double spacing in full width wells in pages with sidebar

Need Review By (Date)

10/09

Urgency

medium

Steps to Test

Collections

  1. In both colorful and traditional reference sites, log in and visit an existing flexible page with a visible sidebar. In that page, create 2 collections with a background color. In one of them select "Limited width" as the value for "Background Color Width", and in the other one select "Full width".
  2. Visit the page and confirm that the horizontal spacing between the well border are the content is the same (32px in traditional and 48px in colorful) for both the "Limited Width" and the "Full Width" versions.
  3. Alternatively, you visit a page with existing collections and change the value of "Background Color Width". Here are some pages that you can use:

Private collections

  1. In the case of the private collections, the test sites already include private pages with "Limited width" and "Full width" example private collections. In both colorful and traditional reference sites, log in and visit the following pages:
  2. In each site, confirm that the horizontal spacing between the well border are the content is the same (32px in traditional and 48px in colorful) for both the "Limited Width" and the "Full Width" versions.
Screenshot 2024-09-30 at 9 55 56 AM

PR Checklist

@cienvaras cienvaras self-assigned this Sep 30, 2024
@ahughes3 ahughes3 temporarily deployed to Tugboat September 30, 2024 04:08 Destroyed
@cienvaras cienvaras marked this pull request as ready for review September 30, 2024 16:22
Copy link
Collaborator

@mariannuar mariannuar left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@cienvaras Looks good!

@cienvaras cienvaras assigned ahughes3 and unassigned cienvaras Oct 2, 2024
@cienvaras cienvaras requested a review from ahughes3 October 2, 2024 15:50
@cienvaras
Copy link
Collaborator Author

@ahughes3 Ready for you!

@ahughes3
Copy link
Collaborator

ahughes3 commented Oct 3, 2024

@cienvaras

  1. Was the update applied to private collections and (regular) collections? If so can you update testing instructions for those cases as well?

  2. As part of the task in ClickUp we also want to add help text to the field in private collections; I'm making a note in CLickUp but it should match the text that's on the field in the regular collection

Applies when the page is full width. If the page includes a sidebar, this option will have no effect.

@ahughes3 ahughes3 temporarily deployed to Tugboat October 3, 2024 14:26 Destroyed
@cienvaras
Copy link
Collaborator Author

@ahughes3

Was the update applied to private collections and (regular) collections? If so can you update testing instructions for those cases as well?

Yes, the styles are shared by both components. I updated the testing instructions

As part of the task in ClickUp we also want to add help text to the field in private collections; I'm making a note in CLickUp but it should match the text that's on the field in the regular collection

Done. I added the help text for the collections in another PR, but forgot the private collections. It's fixed now.

Base automatically changed from 11.2.5-release to develop October 3, 2024 15:39
@ahughes3 ahughes3 requested a review from joegl October 7, 2024 16:14
@ahughes3 ahughes3 assigned joegl and unassigned ahughes3 Oct 7, 2024
@joegl joegl changed the base branch from develop to 11.3.1-release October 7, 2024 18:02
@joegl
Copy link
Contributor

joegl commented Oct 7, 2024

@cienvaras It looks like there's now a conflict based on the merged change in #1640

@ahughes3 ahughes3 temporarily deployed to Tugboat October 7, 2024 23:50 Destroyed
@cienvaras
Copy link
Collaborator Author

@joegl Conflicts fixed, ready to review again.

@joegl joegl merged commit 869439f into 11.3.1-release Oct 9, 2024
17 checks passed
@joegl joegl deleted the shs-5845--full-width-well-extra-spacing branch October 9, 2024 16:04
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