10/UI/Symbol/Glyph add owner, presenter, location, date; update calendar#7
Open
catenglaender wants to merge 1 commit intorelease_10from
Open
10/UI/Symbol/Glyph add owner, presenter, location, date; update calendar#7catenglaender wants to merge 1 commit intorelease_10from
catenglaender wants to merge 1 commit intorelease_10from
Conversation
80117fc to
af26801
Compare
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Issue
While working on the OpenCast plugin, the team noticed the need for some more Glyphs. We believe that these glyphs are a useful addition to ILIAS in general and should be available in the core.
These are the designs of the new/updated glyphs:

We wish to merge these with release_10 so the OpenCast plugin being actively developed for ILIAS 10 can make good use of these icons. They help to make the interface less crowded and core functionality and information easier to find.
Additions
Updates
Beware: Binary files
Font files cannot be merged dynamically by git. If other modifications to the il-icon font files took place since this PR was made these will be overridden.
Outlook
The team recommends moving all Glyphs currently using the Glyphicons-Halflings and the Open Sans Emoji font to their equivalent in the simplelineicons font or to designs created by the community emulating the same look. This way ILIAS will be more visually coherent.
The CSS authorities are discussing moving from icon fonts to single SVGs which would make the hassle of carefully managing and modifying the font files obsolete.