Uploaded image for project: 'Jenkins'
  1. Jenkins
  2. JENKINS-68764

Plugin bundles proprietary font

    XMLWordPrintable

Details

    • 237.v37e2c03964c9

    Description

      Since https://github.com/jenkinsci/embeddable-build-status-plugin/pull/31, the plugin bundles a copy of Verdana, a proprietary font by Microsoft: https://en.wikipedia.org/wiki/Verdana

      Please provide the license allowing distribution of the font, or delete it from the plugin.

      Attachments

        Activity

          basil Basil Crow added a comment -

          Based on my reading of https://github.com/jenkinsci/embeddable-build-status-plugin/blob/8b8dde85476aa491588fa8e8defd7bb36cda5e92/src/main/java/org/jenkinsci/plugins/badge/StatusImage.java#L198-L217 the font is not used to render text but to determine the width of the rendered text. So Verdana itself need not be used, but any open-source font that is metrically compatible with Verdana would suffice. A quick search shows that Bitstream Vera Sans is metrically compatible with Verdana, as is PT Astra Fact. These fonts could be used as a drop-in replacement for Verdana and should yield the same width calculation. (This could be verified in a unit test.)

          basil Basil Crow added a comment - Based on my reading of https://github.com/jenkinsci/embeddable-build-status-plugin/blob/8b8dde85476aa491588fa8e8defd7bb36cda5e92/src/main/java/org/jenkinsci/plugins/badge/StatusImage.java#L198-L217 the font is not used to render text but to determine the width of the rendered text. So Verdana itself need not be used, but any open-source font that is metrically compatible with Verdana would suffice. A quick search shows that Bitstream Vera Sans is metrically compatible with Verdana, as is PT Astra Fact. These fonts could be used as a drop-in replacement for Verdana and should yield the same width calculation. (This could be verified in a unit test.)

          People

            Unassigned Unassigned
            danielbeck Daniel Beck
            Votes:
            0 Vote for this issue
            Watchers:
            2 Start watching this issue

            Dates

              Created:
              Updated:
              Resolved: