Skip to content

NEW: add filter controls for distribution vs additional#30

Merged
gregcaporaso merged 7 commits intomainfrom
non-distro-priority
Mar 18, 2026
Merged

NEW: add filter controls for distribution vs additional#30
gregcaporaso merged 7 commits intomainfrom
non-distro-priority

Conversation

@ebolyen
Copy link
Copy Markdown
Member

@ebolyen ebolyen commented Mar 12, 2026

fixes #28

@cloudflare-workers-and-pages
Copy link
Copy Markdown

cloudflare-workers-and-pages bot commented Mar 12, 2026

Deploying library with  Cloudflare Pages  Cloudflare Pages

Latest commit: 672f747
Status: ✅  Deploy successful!
Preview URL: https://4d98a91b.library-b91.pages.dev
Branch Preview URL: https://non-distro-priority.library-b91.pages.dev

View logs

@github-project-automation github-project-automation bot moved this to Backlog in 2026.4 🌱 Mar 12, 2026
@ebolyen ebolyen moved this from Backlog to Needs Review in 2026.4 🌱 Mar 12, 2026
@gregcaporaso
Copy link
Copy Markdown
Member

This looks good @ebolyen, thank you! One request on nomenclature - what do you think about using "Stand-alone" instead of "Additional", e.g., here:

screenshot 2026-03-17 at 3 37 34 PM

I think "Stand-alone" will be more clear, and it's a term I think I've used in various places to describe non-distro-associated plugins, so I think it will align with language in papers, on the forum, etc. (I would suggest "Distribution-associated" instead of "Distribution" as well, but I'm guessing it's too long of a phrase - but "Distribution-associated" versus "stand-alone" feelings like what we're trying to capture here).

Aside from this, good to go from my perspective - thanks again!

@ebolyen
Copy link
Copy Markdown
Member Author

ebolyen commented Mar 18, 2026

That's a much better term, thanks @gregcaporaso!

Copy link
Copy Markdown
Member Author

@ebolyen ebolyen left a comment

Choose a reason for hiding this comment

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

per @gregcaporaso's review

Comment thread src/lib/contexts/FilterContext.svelte Outdated
Comment thread src/lib/contexts/FilterContext.svelte Outdated
Comment thread src/lib/contexts/index.ts Outdated
Comment thread src/lib/contexts/index.ts Outdated
Comment thread src/routes/plugins/+page.svelte Outdated
Comment thread src/routes/plugins/+page.svelte Outdated
Copy link
Copy Markdown
Member Author

@ebolyen ebolyen left a comment

Choose a reason for hiding this comment

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

distribution-associated definitely looks too long. What about distro-associated?

I also tried in-distribution and from-distribution, but it felt awkward

Comment thread src/lib/contexts/FilterContext.svelte Outdated
Comment thread src/lib/contexts/index.ts Outdated
Comment thread src/routes/plugins/+page.svelte Outdated
@ebolyen
Copy link
Copy Markdown
Member Author

ebolyen commented Mar 18, 2026

@gregcaporaso, feel free to merge either with or without the second round of terms.

gregcaporaso and others added 3 commits March 18, 2026 11:16
Co-authored-by: Evan Bolyen <ebolyen@gmail.com>
Co-authored-by: Evan Bolyen <ebolyen@gmail.com>
Co-authored-by: Evan Bolyen <ebolyen@gmail.com>
@gregcaporaso
Copy link
Copy Markdown
Member

Stand-alone v Distro-associated sounds great. Just applied those suggestions, and this is ready to go after the build is successful. Thanks @ebolyen!

@gregcaporaso gregcaporaso merged commit 0bbdb5d into main Mar 18, 2026
1 check passed
@github-project-automation github-project-automation bot moved this from Needs Review to Changelog Needed in 2026.4 🌱 Mar 18, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Changelog Needed

Development

Successfully merging this pull request may close these issues.

prioritize non-distro-associated plugins on the plugin page

2 participants