-
Notifications
You must be signed in to change notification settings - Fork 729
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
Reworked and cleaned up client-side TabSheet and Accordion. #12357
Conversation
756d067
to
03f61b9
Compare
03f61b9
to
bdc1832
Compare
bdc1832
to
c19811a
Compare
c19811a
to
b4b7412
Compare
- Added and corrected JavaDocs. - Deprecated unused public methods. - Fixed first tab style logic in TabSheet. - Fixed navigation focus logic in TabSheet. - Fixed tab width bookkeeping for scrolling TabSheet tabs. - Renamed private methods and variables for clarity. - Removed unnecessary or duplicated private methods. - Reworked some logic to clarify it and to better match my understanding of what's supposed to happen within those methods. - Updated some deprecated method calls to use currently recommended solutions. - Added and updated regression tests.
b4b7412
to
b3de38a
Compare
@@ -1248,6 +1767,8 @@ private boolean isScrollerAtFirstVisibleTab() { | |||
*/ | |||
@Deprecated | |||
public class PlaceHolder extends VLabel { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ClassCanBeStatic: Inner class is non-static but does not reference enclosing class
(at-me in a reply with help
or ignore
)
TableCellElement spacerCell = ((TableElement) tb.getElement().cast()) | ||
.getRows().getItem(0).getCells().getItem(tb.getTabCount()); | ||
// This element is hidden by Valo, test with legacy themes. | ||
TableCellElement spacerCell = ((TableCellElement) tb.spacerTd.cast()); | ||
if (scroller.getStyle().getDisplay() != "none") { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ReferenceEquality: Comparison using reference equality instead of value equality
(at-me in a reply with help
or ignore
)
of what's supposed to happen within those methods.
solutions.