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.

Sign up for GitHub

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

Jump to bottom

8263472: Specification of JComponent::updateUI should document that the default implementation does nothing #2955

Closed
wants to merge 10 commits into from
Closed

8263472: Specification of JComponent::updateUI should document that the default implementation does nothing #2955

Changes from all commits
Commits
Show all changes
10 commits
Select commit Hold shift + click to select a range
9a0b275
8263472: Specification of JComponent::updateUI doesn't mention that t…
prsadhuk Mar 12, 2021
2d5e911
javadoc change
prsadhuk Mar 15, 2021
a648420
added @implnote
prsadhuk Mar 16, 2021
39c4a72
Use @implNote
prsadhuk Mar 16, 2021
50b997d
@implNote change
prsadhuk Mar 16, 2021
ce72d7b
implNote change
prsadhuk Mar 18, 2021
5b15e87
javadoc change
prsadhuk Mar 18, 2021
36cccda
Modified as per Phil's comment
prsadhuk Mar 22, 2021
ec7755a
Add @implSpec as per JoeDarcy's recommendation
prsadhuk Mar 24, 2021
f48a043
javadoc change
prsadhuk Mar 25, 2021
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Jump to
Jump to file
Failed to load files.
Diff view
Diff view
7 changes: 5 additions & 2 deletions src/java.desktop/share/classes/javax/swing/JComponent.java
View file
Open in desktop
Expand Up @@ -656,15 +656,18 @@ public JComponent() {


/**
* Resets the UI property to a value from the current look and feel.
* <code>JComponent</code> subclasses must override this method
* This method is called to update the UI property to a value from the
* current look and feel.
* {@code JComponent} subclasses must override this method
* like this:
* <pre>
* public void updateUI() {
* setUI((SliderUI)UIManager.getUI(this);
* }
* </pre>
*
* @implSpec The default implementation of this method does nothing.
*
prsadhuk marked this conversation as resolved.
Show resolved Hide resolved
* @see #setUI
* @see UIManager#getLookAndFeel
* @see UIManager#getUI
Expand Down