Skip to content

Updated Belize flags - #1384

Open
p2made wants to merge 5 commits into
lipis:mainfrom
p2made:Belize
Open

Updated Belize flags#1384
p2made wants to merge 5 commits into
lipis:mainfrom
p2made:Belize

Conversation

@p2made

@p2made p2made commented Jun 19, 2025

Copy link
Copy Markdown
Contributor

This PR corrects the Belize flags as per #1315 & replaces #1372.

The files have been optimised using...

svgo --multipass --pretty --indent 2 -q -i ./source/bz-1x1.svg -o ./flag-icons/flags/1x1/bz.svg
svgo --multipass --pretty --indent 2 -q -i ./source/bz-4x3.svg -o ./flag-icons/flags/4x3/bz.svg

Using yarn removes a lot of detail.

Comment thread sass/_variables.scss
// Keep xx first as it is often used as fallback
"xx",

// ISO Countries and Places

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

IMO I would keep these in separate PRs...leave this one for strictly updating the Belize flag

Comment thread flags/1x1/bz.svg
<path d="m229.1 307.4 11-10.7-.5 6.4s-5.7 3.6-5.7 6.2l-4.8-2m5.9 2.3s1.1-3.4 4.2-5l-.3 6.6-4-1.7zm15.7-18.5s4.5-.6 7.8 0c0 0-.5 4.5 0 5.3 0 0-6.1-1.4-8.6.9 0 0-.9-4.2.8-6.2zm-9.5 1.1s4.5-.6 7.8 0c0 0-.8 5-.3 5.9 0 0-5-2.8-7.5-.5 0 0-1.6-3.5 0-5.4z"/>
<path d="M240.9 298s4.5-.4 7.8 0c0 0-.5 4.6 0 5.4 0 0-5.8-1.4-8.3.8 0 0-1.2-4.1.5-6.1zm9-.8s6.4-.8 9.8-.3c0 0-1.4 3.7.3 6.2 0 0-7-1.9-9.6.3 0 0-2.2-4.2-.5-6.1zm-9.5 7 8.6.7s-.8 6.3-.3 7.2c0 0-7-1.7-8.3.6 0 0-2.3-5.9 0-8.5zm9.8 0s7.5-.5 11 0c0 0-2 4.8.1 9 0 0-9.8-2.8-11.1-.5 0 0-2.3-5.9 0-8.5zm11.4-13.1s4.5-.6 7.9 0c0 0-.6 4.5 0 5.3 0 0-6.2-1.4-8.7.9 0 0-.8-4.2.8-6.2zm0 6.2s4.5-.6 7.9 0c0 0-.6 4.4 0 5.3 0 0-6.2-1.4-8.7.8 0 0-.8-4.2.8-6.1zm.6 6.4s4.4-.6 7.9 0c0 0-1.2 7-.6 7.8 0 0-5.1-2.4-7.5-.3 0 0-1.5-5.5.2-7.5z"/>
<path d="M271.4 304.2s1.5 5.3 3.4 6l-2.5 3s-3-2.2-3.4-4.7c0 0 2-2.3 2.5-4.3z"/>
<svg xmlns="http://www.w3.org/2000/svg" id="svg57441" width="512" height="512" version="1.1">

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Suggested change
<svg xmlns="http://www.w3.org/2000/svg" id="svg57441" width="512" height="512" version="1.1">
<svg xmlns="http://www.w3.org/2000/svg" xmlns:xlink="http://www.w3.org/1999/xlink" id="flag-icons-bz" viewBox="0 0 512 512">

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

This same style of change should be applied to the 4x3 resolution as well

Comment thread flags/1x1/bz.svg
<path d="M240.9 298s4.5-.4 7.8 0c0 0-.5 4.6 0 5.4 0 0-5.8-1.4-8.3.8 0 0-1.2-4.1.5-6.1zm9-.8s6.4-.8 9.8-.3c0 0-1.4 3.7.3 6.2 0 0-7-1.9-9.6.3 0 0-2.2-4.2-.5-6.1zm-9.5 7 8.6.7s-.8 6.3-.3 7.2c0 0-7-1.7-8.3.6 0 0-2.3-5.9 0-8.5zm9.8 0s7.5-.5 11 0c0 0-2 4.8.1 9 0 0-9.8-2.8-11.1-.5 0 0-2.3-5.9 0-8.5zm11.4-13.1s4.5-.6 7.9 0c0 0-.6 4.5 0 5.3 0 0-6.2-1.4-8.7.9 0 0-.8-4.2.8-6.2zm0 6.2s4.5-.6 7.9 0c0 0-.6 4.4 0 5.3 0 0-6.2-1.4-8.7.8 0 0-.8-4.2.8-6.1zm.6 6.4s4.4-.6 7.9 0c0 0-1.2 7-.6 7.8 0 0-5.1-2.4-7.5-.3 0 0-1.5-5.5.2-7.5z"/>
<path d="M271.4 304.2s1.5 5.3 3.4 6l-2.5 3s-3-2.2-3.4-4.7c0 0 2-2.3 2.5-4.3z"/>
<svg xmlns="http://www.w3.org/2000/svg" id="svg57441" width="512" height="512" version="1.1">
<style id="style132">

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

This shouldn't have style IDs like this, they should be optimized down. If you just run svgo from the root folder without passing anything in, it will take the config defined within the project structure that should optimize this to the "standards" of the project.

@NotTsunami

Copy link
Copy Markdown
Collaborator

If you address the comments above, I can merge this!

@NotTsunami NotTsunami self-assigned this Apr 4, 2026
@NotTsunami

Copy link
Copy Markdown
Collaborator

If there are no updates to this - I will cherry-pick (retaining original authorship) and apply the requested changes to a new pull request. Thanks!

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.

2 participants