Skip to content

Create "Column" and "ThemedColumn" components and apply them across the website.#535

Merged
brian-montgomery merged 31 commits into
mainfrom
alexis/column-components
Apr 14, 2026
Merged

Create "Column" and "ThemedColumn" components and apply them across the website.#535
brian-montgomery merged 31 commits into
mainfrom
alexis/column-components

Conversation

@alexisreytech

Copy link
Copy Markdown
Contributor

Fixes #430 and applies the new "Column" and "ThemedColumn" components across the website. The "ThemedColumn" was mostly able to be used in place of where the "ThemedBox" was being used, but in some cases the "ThemedBox" was better suited.

alexisreytech and others added 30 commits March 31, 2026 15:53
….astro.

Co-authored-by: brian-montgomery <brian-montgomery@users.noreply.github.com>
@netlify

netlify Bot commented Apr 9, 2026

Copy link
Copy Markdown

Deploy Preview for accessiblecommunity ready!

Name Link
🔨 Latest commit 314f781
🔍 Latest deploy log https://app.netlify.com/projects/accessiblecommunity/deploys/69d92e7afed6030008242ef1
😎 Deploy Preview https://deploy-preview-535--accessiblecommunity.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@brian-montgomery brian-montgomery left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Thank you for finding and changing all of these, Alexis. I'm sure that took some time.

}

.bg-example {
background-image: url(${fixBgNoColor.src});

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Just a note: I don't think this fix will work (in this case), but it's okay, because this page isn't being used (and we should probably remove it).

export interface Props {
id?: string;
class?: string;
noDefault?: boolean;

@brian-montgomery brian-montgomery Apr 14, 2026

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

I still don't love the noDefault name we came up with, but I still can't think of anything better.

Perhaps the right thing to do is to have it be two separate Components, one with the col (or an appropriately sized col) and one without? I don't know.

@brian-montgomery
brian-montgomery merged commit 9ec0e41 into main Apr 14, 2026
9 checks passed
@brian-montgomery
brian-montgomery deleted the alexis/column-components branch April 14, 2026 15:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Component: Column

2 participants