LPS-35668 fix bug introduced when migrating toolbar
LPS-35668 encapsulate javascript code to avoid duplication
LPS-35668 - Source formatting
LPS-35401 Fix source formatting

We have to use the full package name because the IconHelpTag already imported is not the class that we want to use.
LPS-35401 iconHelp and iconHelp(String) shouldn't give 2 different results, because that's not immediately obvious. Overriding methods usually changes behaviors of the same method, not different methods. Rename icon* to portletIcon*, and keep iconHelp(String) as the method to match "icon" from the "liferay-ui" namespace
LPS-35401 Update old references
LPS-35401 Nate, back in the day, liked the name "theme" vs "taglibLiferay", but both reference the same VelocityTaglib obj
LPS-35747 The global site should be considered a system site so that it cannot be deleted.
LPS-35747 Disable the checkbox if the site cannot be deleted
LPS-35747 Remove the "Delete" item from the actions menu for the system sites.
LPS-34850 Re-add old Javadoc to updated code
LPS-34850 Edit Javadoc
LPS-34850 Edit Javadoc
LPS-34850 Javadoc - Service implementation class descriptions should summarize actions that can be done on the respective entities
LPS-34850 Autogenerated
LPS-34850 No need for the @see
LPS-35804 PanelTag should clean up attributes on doEndTag
LPS-35804 Set extended to null on clean up
LPS-35804 Avoid Boolean vs. boolean. The other properties in these tags, like "accordion" or "persistState" are boolean and not Boolean. We should keep it consistent. The bug is that we never reset it to false on clean up.
LPS-35037 Remove unused method