-
-
Notifications
You must be signed in to change notification settings - Fork 8.6k
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
Improve log record icon #7217
Improve log record icon #7217
Conversation
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.
I don't think the backside of a bank card is quite suitable to access Jenkins' logs 🤔
Good thing, then, that I am not proposing to introduce the back side of a bank card. I am proposing to introduce a journal: https://www.amazon.com/Moleskine-Classic-Cover-Notebook-Ruled/dp/8883701127 |
Note that ionicons has a separate |
Looks fine to me, and apparently to the ionicon developers as well, because otherwise they would not have created distinct
I do not see any reason why it would. |
As far as I can tell there is no objection to the journal concept, just a dislike of the appearance of the |
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.
sure let's try it
Holding back a review is not my intention 👀 This PR is now ready for merge. We will merge it after ~24 hours if there is no negative feedback. |
Co-authored-by: Basil Crow <me@basilcrow.com>
A full file tray icon is used for build history, as seen here. An empty file tray is used for log records. Apart from raising confusing questions about why one tray is empty and the other is full, I do not think it makes sense to use the same icon for both things: one is a historical record of a build, which very much seems like the type of thing you would "file away" for future reference in the physical world, but the system log is a short-term record of events, more analogous to a journal in the physical world. Indeed, the logging framework in
systemd
is calledsystemd-journald
, and journaling file systems have been around for decades.Before
and
After
and
Proposed changelog entries
Proposed upgrade guidelines
N/A
Submitter checklist
@Restricted
or have@since TODO
Javadocs, as appropriate.@Deprecated(since = "TODO")
or@Deprecated(forRemoval = true, since = "TODO")
, if applicable.eval
to ease future introduction of Content Security Policy (CSP) directives (see documentation).Desired reviewers
@mention
Maintainer checklist
Before the changes are marked as
ready-for-merge
:upgrade-guide-needed
label is set and there is a Proposed upgrade guidelines section in the pull request title (see example).lts-candidate
to be considered (see query).