-
Notifications
You must be signed in to change notification settings - Fork 84
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
Fix tile titles #198
Fix tile titles #198
Conversation
Setting the source will change the title, but changing the title no longer affects the source name
Codecov Report
@@ Coverage Diff @@
## master #198 +/- ##
============================================
- Coverage 15.12% 15.05% -0.08%
Complexity 241 241
============================================
Files 149 150 +1
Lines 3834 3853 +19
Branches 290 290
============================================
Hits 580 580
- Misses 3223 3242 +19
Partials 31 31 |
@@ -48,7 +48,6 @@ public EditableLabel() { | |||
if (isEditing) { | |||
editField.requestFocus(); | |||
} | |||
label.setVisible(!isEditing); |
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.
?
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.
The visible property is already bound so all this does is throw an exception
log.log(Level.SEVERE, "Could not get the tab containing the widget pane!", e); | ||
} | ||
} | ||
|
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.
from discussion: let's leave this off for now until we can refactor auto-population
Sets the text overrun to the left side to make titles look like
...system/Victor/Speed
instead of/LiveWindow/MySub...
Uses the tab source prefix to remove redundant leading information
Changing the widget title no longer changes the name of its source
Fixes #29