Skip to content

New Iconset#230

Merged
koppor merged 49 commits into
masterfrom
icons
Oct 25, 2015
Merged

New Iconset#230
koppor merged 49 commits into
masterfrom
icons

Conversation

@stefan-kolb

Copy link
Copy Markdown
Member

Also see #130

  • Unlike expected, PNG files do not look sharper than the font version for me
  • Anyhow, we should go up to 24px as 16px seems way to small
  • Strangely, some icons are missing in the materialize Github repository, e.g. settings not relevant
  • We still have duplicated and unused icons in the properties file
  • Open file icon should be either file-pdf or file-document (or attachment) depending on actual file type.
  • How are icons for external file types handled? users cannot set this anymore, they have to use the default icons instead.
  • squash commits so that temporary binary files are not in the history
  • separator are not removed when the icons are hidden (see special fields like rank)
  • add information about used font into external-libraries.txt and update "About" content

@stefan-kolb stefan-kolb added this to the v2.80 milestone Oct 12, 2015
@koppor

koppor commented Oct 12, 2015

Copy link
Copy Markdown
Member

When checking for the size of the icons, please also check high resolutions settings:

@koppor koppor mentioned this pull request Oct 12, 2015
@matthiasgeiger

Copy link
Copy Markdown
Member

So we should use the Font as this enables simple resizing.
I will investigate whether it is possible to make the icons sharper somehow...

@kubovy

kubovy commented Oct 14, 2015

Copy link
Copy Markdown
Contributor

What about some icon dpi scheme ala android: mdpi, hdpi, xhdpi, ...? Since resolution are growing. If you want to use vector format what about SVG?

@simonharrer

Copy link
Copy Markdown
Contributor

We wanted to use an icon font. This should scale for all resolutions..

@simonharrer

Copy link
Copy Markdown
Contributor

What needs to be investigated

  • the menu item gets bigger when we use the 24px icon. This looks strange when we have normal menu items with no icons and menu items with the 24px icons in a single menu.
  • disabled icons look strange

ideas

  • the selection of the color to mark an entry should be done in the tool bar as well right next the marker symbol - or we change the color symbol in the toolbar depending on the current color selection.

@simonharrer

Copy link
Copy Markdown
Contributor

👍 for linking to the donations page as it also explains how the money is spent.

@matthiasgeiger

Copy link
Copy Markdown
Member

Okay... I think we are coming down the stretch...

@koppor @lenhard (and all interested others): Do you have some feedback to be integrated before we merge into master?

@simonharrer

Copy link
Copy Markdown
Contributor

Squashed the commits so that temporary png files and fonts are pruned from the history. And removed the ability to change the icon of an external file type entry. Hence, only a single item in this PR is remaining.

@koppor

koppor commented Oct 25, 2015

Copy link
Copy Markdown
Member

The last issue was partially addressed by e57fb49. If #257 goes through, this PR can be merged as is.

koppor added a commit that referenced this pull request Oct 25, 2015
@koppor koppor merged commit a5bb73c into master Oct 25, 2015
@koppor koppor deleted the icons branch October 26, 2015 00:06
koppor pushed a commit that referenced this pull request Jan 15, 2022
4a551a87c3 copied .github/workflows/sheldon.yaml from styles
210d56d1a3 copied .github/workflows/sheldon.yaml from styles
684ab7c021 copied .github/workflows/sheldon.yaml from styles
3313747417 Update Sheldon to csl-styles 2.0
d568b6b3df copied .github/workflows/sheldon.yaml from styles
96d7471411 copied .github/workflows/sheldon.yaml from styles
65417b390f copied .github/workflows/sheldon.yaml from styles
841eaef79f copied .github/workflows/sheldon.yaml from styles
5369c686fd copied .github/workflows/sheldon.yaml from styles
ab41f49d31 copied .github/workflows/sheldon.yaml from styles
dfbd6e4f0f copied .github/workflows/sheldon.yaml from styles
0c025165bd copied .github/workflows/sheldon.yaml from styles
511a577c5a copied .github/workflows/sheldon.yaml from styles
b963638d37 copied .github/workflows/sheldon.yaml from styles
45813b71b1 add new terms for 1.0.2 (#230)
4b179c7ce8 copied .github/workflows/merge.yaml from styles
86b9844aba update to csl 2.0
18458dcf86 bundle update

git-subtree-dir: buildres/csl/csl-locales
git-subtree-split: 4a551a87c30db341809dd203765cd8f29c070b36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants