• Icon: Epic Epic
    • Resolution: Unresolved
    • Icon: Minor Minor
    • core
    • Remove YUI

      Jenkins currently uses YUI version 2.9.0 released in 2011. There is no newer version of YUI 2.x. YUI3 is available but was also set to deprecated in 2014.

      The YUI library is heavy weight and the size of all loaded javascript is over 400k

      The YUI library is hardly used in core and activities are already ongoing to remove the last usages of it.

      Plugins should move away from using YUI and replace calls to YUI with plain javascript. In most cases the effort is rather small.

      The complete list of plugins that will be affected by the removal of YUI is found here: https://docs.google.com/spreadsheets/d/1UjvtFmNmEdjMN5DUoFxJfBryA8q-E5_HwOzVKbVG9b0/edit?usp=sharing

          [JENKINS-73539] Remove YUI

          Basil Crow added a comment -

          Found a usage in branch-api as well:

          src/main/resources/jenkins/branch/PropertyMigration/MonitorImpl/message.jelly
          20:                    <span class="yui-button yui-link-button primary">
          35:                  <span class="yui-button yui-link-button">
          71:                    <span class="yui-button yui-link-button" name="dynamicLoad">
          99:                    <span class="yui-button yui-link-button primary" name="${p.pluginUpgrade?'Submit':'dynamicLoad'}">
          

          Basil Crow added a comment - Found a usage in branch-api as well: src/main/resources/jenkins/branch/PropertyMigration/MonitorImpl/message.jelly 20: <span class= "yui-button yui-link-button primary" > 35: <span class= "yui-button yui-link-button" > 71: <span class= "yui-button yui-link-button" name= "dynamicLoad" > 99: <span class= "yui-button yui-link-button primary" name= "${p.pluginUpgrade? 'Submit' : 'dynamicLoad' }" >

          Markus Winter added a comment -

          This is not really making use of yui. That's just some css styling applied, it doesn't lead to loading yui libraries.

          Markus Winter added a comment - This is not really making use of yui. That's just some css styling applied, it doesn't lead to loading yui libraries.

          Basil Crow added a comment -

          Presumably both YUI JavaScript and CSS should be removed.

          Basil Crow added a comment - Presumably both YUI JavaScript and CSS should be removed.

          Mark Waite added a comment - - edited

          https://www.jenkins.io/changelog/2.490/ disables YahooUI by default but does not remove it. I've left this issue open until we can remove it.

          Mark Waite added a comment - - edited https://www.jenkins.io/changelog/2.490/ disables YahooUI by default but does not remove it. I've left this issue open until we can remove it.

          Basil Crow added a comment -

          Presumably both YUI JavaScript and CSS should be removed.

          Done in jenkinsci/branch-api-plugin#498.

          Basil Crow added a comment - Presumably both YUI JavaScript and CSS should be removed. Done in jenkinsci/branch-api-plugin#498 .

            Unassigned Unassigned
            mawinter69 Markus Winter
            Votes:
            0 Vote for this issue
            Watchers:
            3 Start watching this issue

              Created:
              Updated: