Skip to content

Conversation

@MarkEWaite
Copy link
Contributor

@MarkEWaite MarkEWaite commented Jan 5, 2025

Replace outdated icons with analytics symbol

The design library shows the preferred images for Jenkins icons. The symbols scale better and look more consistent with the rest of Jenkins.

No tests added because this a a look and feel change. Not well suited to automated tests.

What has been done

  1. I chose "analytics" from the available symbols listed in Jenkins core.
  2. I added the page-headline class to the h1 headings because that seems to be a class that is used in other page headlines.

Screenshots

Before After
before after

How to test

  • Confirmed that images were displayed as expected in freestyle projects and in matrix projects

Checklist

  • Git commits follow best practices
  • Build passes in Jenkins
  • Appropriate tests or explanation to why this change has no tests
  • Pull Request is marked with appropriate label (see .github/release-drafter.yml)
  • For dependency updates: links to external changelogs and, if possible, full diffs

https://weekly.ci.jenkins.io/design-library/symbols/ shows the preferred
images for Jenkins icons.  The symbols scale better and look more
consistent with the rest of Jenkins.

I chose "analytics" from the available symbols listed at
https://github.com/jenkinsci/jenkins/tree/master/war/src/main/resources/images/symbols

I added the page-headline class to the h1 headings because that seems
to be a class that is used in other page headlines.

Testing done:

* Confirmed that images were displayed as expected in freestyle projects
  and in matrix projects
@MarkEWaite MarkEWaite added the bug Bug fix label Jan 5, 2025
@jonesbusy
Copy link
Contributor

Thanks!

@gounthar gounthar merged commit e862949 into jenkinsci:master Jan 5, 2025
17 checks passed
@MarkEWaite MarkEWaite deleted the replace-icons-with-symbols branch January 5, 2025 12:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Bug fix

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants