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

docs: introduce grid guidelines #1705

Merged
merged 9 commits into from
Jun 12, 2024
Merged

docs: introduce grid guidelines #1705

merged 9 commits into from
Jun 12, 2024

Conversation

frshwtr
Copy link
Contributor

@frshwtr frshwtr commented Jun 11, 2024

Describe your changes

Issue ticket number and link

Link to Figma designs

Checklist before requesting a review

  • I have applied percy label for o-[COMPONENT] or chromatic label for o3-[COMPONENT] on my PR before merging and after review. Find more details in CONTRIBUTING.md
  • If it is a new feature, I have added thorough tests.
  • I have updated relevant docs.
  • I have updated relevant env variables in Doppler.

@frshwtr frshwtr requested a review from a team as a code owner June 11, 2024 12:38
@origamiserviceuser origamiserviceuser added this to Backlog in Origami ✨ Jun 11, 2024
@notlee notlee temporarily deployed to origami-webs-or-705-int-avwr7m June 11, 2024 12:50 Inactive

<Guideline summary="Aligning content to grid" do={true}>
<p slot="description">
Create a container around the sub-items in a page e.g ads banners to align with the grid columns, avoid having a
Copy link
Contributor

Choose a reason for hiding this comment

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

I'm struggling to understand what this means, could it be reworded for clarity or perhaps enhanced with a visual aide available?

@notlee notlee temporarily deployed to origami-webs-or-705-int-avwr7m June 11, 2024 15:53 Inactive
@notlee notlee temporarily deployed to origami-webs-or-705-int-avwr7m June 12, 2024 10:26 Inactive
@notlee notlee temporarily deployed to origami-webs-or-705-int-avwr7m June 12, 2024 10:43 Inactive
@notlee notlee temporarily deployed to origami-webs-or-705-int-avwr7m June 12, 2024 10:44 Inactive
@notlee notlee temporarily deployed to origami-webs-or-705-int-avwr7m June 12, 2024 10:55 Inactive
@notlee notlee temporarily deployed to origami-webs-or-705-int-avwr7m June 12, 2024 10:57 Inactive
It is a design foundation, not a component
@notlee notlee temporarily deployed to origami-webs-or-705-int-avwr7m June 12, 2024 12:45 Inactive
@notlee notlee merged commit 36a3f4f into main Jun 12, 2024
7 checks passed
Origami ✨ automation moved this from Backlog to Done Jun 12, 2024
@notlee notlee deleted the or-705/introduce-grid-guides branch June 12, 2024 12:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Origami ✨
  
Done
Development

Successfully merging this pull request may close these issues.

None yet

2 participants