Skip to content

Adding request layout POCs to scaffolding #2488

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

Draft
wants to merge 40 commits into
base: master
Choose a base branch
from

Conversation

taylorvnoj
Copy link
Contributor

Motivations

Changes

Added

Changed

Deprecated

Removed

Fixed

Security

Testing

Changes can be
tested via Pre-release


In Atlantis we use Github's built in pull request reviews.

Random photo of Atlantis

* removing rectangle a few other things

* adding layout adjustments based on real-world scenarios

* removing console logs

* replacing the name Center for ContentBlock

* modified to remove dev ovverrides
scotttjob and others added 10 commits April 10, 2025 15:27
* updating space for ga

* semantic tags in the layout components

* allowing passthrough of data, aria, id, and role.

* unsafe styles and classnames. no more media queries, prop passthrough, semantic elements

* final cleanup on layout addressing

* adding some additional feedback

* actually making these new props work

* readding collapse below

* adding missing styles

* updating common types to be more specific

* making types a bit more specific and adding js doc comments for them

* updating Cover.Center to follow the same patterns as the rest

* adding better types

* removing unnecessary processing step

* removing unnecessary processing step

* updating prop name

* Add andText to story default args

* adding the latest item

---------

Co-authored-by: Jacob Deichert <[email protected]>
Copy link

⚠️ Pull Request titles in this repo follow the Conventional Commits specification.

No release type found in pull request title "Adding request layout POCs to scaffolding". Add a prefix to indicate what kind of release this pull request corresponds to. For reference, see https://www.conventionalcommits.org/

Available types:

  • feat: A new feature
  • fix: A bug fix
  • docs: Documentation only changes
  • style: Changes that do not affect the meaning of the code (white-space, formatting, missing semi-colons, etc)
  • refactor: A code change that neither fixes a bug nor adds a feature
  • perf: A code change that improves performance
  • test: Adding missing tests or correcting existing tests
  • build: Changes that affect the build system or external dependencies (example scopes: gulp, broccoli, npm)
  • ci: Changes to our CI configuration files and scripts (example scopes: Travis, Circle, BrowserStack, SauceLabs)
  • chore: Other changes that don't modify src or test files
  • revert: Reverts a previous commit

Copy link

Deploying atlantis with  Cloudflare Pages  Cloudflare Pages

Latest commit: 0923efe
Status: ✅  Deploy successful!
Preview URL: https://bb2f4fbb.atlantis.pages.dev
Branch Preview URL: https://taylor-scaffolding-objects.atlantis.pages.dev

View logs

Base automatically changed from scott_t/layouts to master April 16, 2025 09:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants