Skip to content
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

Progressively get rid of YUI #1599

Open
ebruchez opened this issue Mar 12, 2014 · 3 comments
Open

Progressively get rid of YUI #1599

ebruchez opened this issue Mar 12, 2014 · 3 comments

Comments

@ebruchez
Copy link
Collaborator

@ebruchez ebruchez commented Mar 12, 2014

We now use jQuery, Bootstrap, and other client-side components like TinyMCE. We should be able to retire YUI.

Actions:

  • move to jQuery
    • DOM manipulation (YAHOO.util.Dom) and base utilities (counts of occurrences under resources-packaged/ops/javascript as of 2016-03-20)
      • 33 YAHOO.util.Dom.addClass
      • 30 YAHOO.util.Dom.removeClass
      • 20 YAHOO.util.Dom.hasClass
      • 19 YAHOO.util.Dom.getElementsByClassName
      • 7 YAHOO.util.Dom.setStyle
      • 7 YAHOO.util.Dom.generateId
      • 4 YAHOO.util.Dom.inDocument
      • 4 YAHOO.util.Dom.getChildren
      • 3 YAHOO.util.Dom.getPreviousSibling
      • 3 YAHOO.util.Dom.getNextSibling
      • 3 YAHOO.util.Dom.getFirstChild
      • 3 YAHOO.util.Dom.getAncestorByClassName
      • 2 YAHOO.util.Dom.insertBefore
      • 2 YAHOO.util.Dom.getViewportHeight
      • 1 YAHOO.util.Dom.insertAfter
      • 1 YAHOO.util.Dom.getViewportWidth
      • 1 YAHOO.util.Dom.getStyle
      • 1 YAHOO.util.Dom.getRegion
      • 1 YAHOO.util.Dom.getElementsBy
      • 1 YAHOO.util.Dom.getDocumentScrollTop
      • 1 YAHOO.util.Dom.getDocumentScrollLeft
    • more DOM manipulation (ORBEON.util.Dom) (counts of occurrences under resources-packaged/ops/javascript as of 2016-03-20)
      • 74 ORBEON.util.Dom.getAttribute
      • 61 ORBEON.util.Dom.get
      • 22 ORBEON.util.Dom.getChildElementByIndex
      • 13 ORBEON.util.Dom.getStringValue
      • 11 ORBEON.util.Dom.getChildElementByClass
      • 8 ORBEON.util.Dom.getElementByTagName
      • 7 ORBEON.util.Dom.isElement
      • 5 ORBEON.util.Dom.getElementsByName
      • 4 ORBEON.util.Dom.setStringValue
      • 2 ORBEON.util.Dom.stringToDom
      • 2 ORBEON.util.Dom.setAttribute
      • 2 ORBEON.util.Dom.nudgeAfterDelay
      • 2 ORBEON.util.Dom.focus
      • 2 ORBEON.util.Dom.blur
      • 1 ORBEON.util.Dom.existsAncestorOrSelf
      • 1 ORBEON.util.Dom.ELEMENT_TYPE
    • some event handling
    • Ajax calls (YAHOO.util.Connect )
  • move to Bootstrap
    • dialogs (#2835)
    • tooltips [2016-06-01: seems we are still using YUI tooltips if oxf.fr.detail.hints.inline is set to false]
  • keep but move YUI dependencies to the XBL component
    • fr:autocomplete
      - 2019-12-10: We have an Improved autocomplete control (#1433) but fr:autocomplete remains for backward compatibility
  • later
    • Improved autocomplete control #1433
  • replace YUI calendar #2800
    • 2019-12-10: We use another calendar as part of fr:date but YUI calendar is still in for backward compatibility.
  • remove altogether
    • YUI RTE support
    • ProgressBar

Any other?

@ebruchez ebruchez added this to the 4.x (to review) milestone Mar 12, 2014
@ebruchez
Copy link
Collaborator Author

@ebruchez ebruchez commented Oct 1, 2014

@avernet avernet removed this from the Review milestone Oct 17, 2014
@ebruchez ebruchez removed this from the Review milestone Oct 17, 2014
@ebruchez ebruchez mentioned this issue Jul 9, 2015
1 of 2 tasks complete
@ebruchez
Copy link
Collaborator Author

@ebruchez ebruchez commented Mar 20, 2016

Removed RTE for 4.11 with fix to #2618: see c82604a and 67f7883.

@ebruchez
Copy link
Collaborator Author

@ebruchez ebruchez commented Apr 30, 2020

Some of the steps above might be outdated. For example:

  • we use jQuery less and aim for even less
  • for dialogs and tooltips, we have to consider whether we want to depend on Bootstrap
    • for example look at Vex
ebruchez pushed a commit that referenced this issue May 16, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
2 participants