-
Notifications
You must be signed in to change notification settings - Fork 90
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
Polish iconography #4391
base: main
Are you sure you want to change the base?
Polish iconography #4391
Conversation
|
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
4bd1536
to
8344a36
Compare
@amritadesmet how do we feel about splitting into two pages ? |
8344a36
to
c625fd2
Compare
c625fd2
to
b5dedaf
Compare
@mark-tate Happy with splitting this - looks good to me now! |
}, [filteredSymbols, deferredSearch]); | ||
|
||
return ( | ||
<Suspense fallback="Loading..."> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Apart from fixing lint, this takes quite long to load, do you want a different / better loading message ? e.g. using the spinner component?
key={countryCode} | ||
> | ||
<LazyCountrySymbol code={countryCode} size={1} /> | ||
<Text>{countryCode}</Text> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Since looking, do you feel this needs to be bolder?
1d1d5cc
to
ed679e7
Compare
- split into 2 pages, icons and country symbols, to aid readability - added filters to icon preview, so you can show/hide solid/outline icons - aligned the country symbol preview component with the icon preview (clear button etc, no results view)
ed679e7
to
72e7a20
Compare
layout: DetailTechnical | ||
--- | ||
|
||
Country symbols represent a specific country and are used in contexts where geographical identification is necessary. They can be considered icons that are visually representative of a country or inter-governmental organisation. All flags are available in both circle and rectangle shapes. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Do we want to use the term flags?
|
||
<Image | ||
src="/img/foundations/iconography-countrysymbols-variant.png" | ||
alt="Example of circular flag next to rectangular flag" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Are they symbols or flags
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Good point thanks, changed it!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@amritadesmet There are other references to flag
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Changed the alt text.
No description provided.