[PR #402] [MERGED] Choose flags to include #783

Closed
opened 2026-03-03 14:40:43 +03:00 by kerem · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/lipis/flag-icons/pull/402
Author: @rctneil
Created: 10/22/2017
Status: Merged
Merged: 10/26/2018
Merged by: @lipis

Base: masterHead: master


📝 Commits (4)

  • a18bb66 Added option to only include a small set of flags
  • b9c2d2f Move all flag entries into one and more changes
  • dec1a30 Update sass/_flag-icon-list.scss
  • 9a547a7 Update _flag-icon-list.scss

📊 Changes

3 files changed (+264 additions, -257 deletions)

View changed files

📝 sass/_flag-icon-list.scss (+264 -249)
sass/_flag-icon-more.scss (+0 -7)
📝 sass/flag-icon.scss (+0 -1)

📄 Description

Give an option to allow users to specify which flags to include in the output CSS.

Specify using a variable list in SCSS like so:

$include: 'as', 'bl', 'si';

If this variable is omitted then all flags are imported as normal.

Also moved the "more" flags into the main list as it seems pointless separating them. Maybe this could also be expanded to have an $exclude variable too so it includes them all apart from the ones specified. It then would allow full control in three different ways!

  • All flags
  • All but some (exclude)
  • Some but not all (include)

Thanks,
Neil


🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.

## 📋 Pull Request Information **Original PR:** https://github.com/lipis/flag-icons/pull/402 **Author:** [@rctneil](https://github.com/rctneil) **Created:** 10/22/2017 **Status:** ✅ Merged **Merged:** 10/26/2018 **Merged by:** [@lipis](https://github.com/lipis) **Base:** `master` ← **Head:** `master` --- ### 📝 Commits (4) - [`a18bb66`](https://github.com/lipis/flag-icons/commit/a18bb66ca1593514147ebb98099bf9dc52f85019) Added option to only include a small set of flags - [`b9c2d2f`](https://github.com/lipis/flag-icons/commit/b9c2d2f5815723f9226e0fe8bf10690c3300b661) Move all flag entries into one and more changes - [`dec1a30`](https://github.com/lipis/flag-icons/commit/dec1a30baa517e58c7f17cc46d6edcd8f6d60da2) Update sass/_flag-icon-list.scss - [`9a547a7`](https://github.com/lipis/flag-icons/commit/9a547a7d29e91187522ccb0dfae3a049ad3766c7) Update _flag-icon-list.scss ### 📊 Changes **3 files changed** (+264 additions, -257 deletions) <details> <summary>View changed files</summary> 📝 `sass/_flag-icon-list.scss` (+264 -249) ➖ `sass/_flag-icon-more.scss` (+0 -7) 📝 `sass/flag-icon.scss` (+0 -1) </details> ### 📄 Description Give an option to allow users to specify which flags to include in the output CSS. Specify using a variable list in SCSS like so: `$include: 'as', 'bl', 'si';` If this variable is omitted then all flags are imported as normal. Also moved the "more" flags into the main list as it seems pointless separating them. Maybe this could also be expanded to have an $exclude variable too so it includes them all apart from the ones specified. It then would allow full control in three different ways! - All flags - All but some (exclude) - Some but not all (include) Thanks, Neil --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>
kerem 2026-03-03 14:40:43 +03:00
Sign in to join this conversation.
No milestone
No project
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
starred/flag-icons#783
No description provided.