Revision: 7678
http://languagetool.svn.sourceforge.net/languagetool/?rev=7678&view=rev
Author: dnaber
Date: 2012-07-08 12:07:40 +0000 (Sun, 08 Jul 2012)
Log Message:
-----------
GUI usability: show wait cursor when enabling/disabling rules
Modified Paths:
--------------
trunk/JLanguageTool/src/java/org/languagetool/gui/Main.java
trunk/JLanguageTool/src/java/org/languagetool/gui/ResultArea.java
Modified: trunk/JLanguageTool/src/java/org/languagetool/gui/Main.java
===================================================================
--- trunk/JLanguageTool/src/java/org/languagetool/gui/Main.java 2012-07-08
11:49:27 UTC (rev 7677)
+++ trunk/JLanguageTool/src/java/org/languagetool/gui/Main.java 2012-07-08
12:07:40 UTC (rev 7678)
@@ -461,6 +461,7 @@
langName = lang.getTranslatedName(messages);
}
new Thread() {
+ @Override
public void run() {
if (!isAlreadyChecking) {
isAlreadyChecking = true;
Modified: trunk/JLanguageTool/src/java/org/languagetool/gui/ResultArea.java
===================================================================
--- trunk/JLanguageTool/src/java/org/languagetool/gui/ResultArea.java
2012-07-08 11:49:27 UTC (rev 7677)
+++ trunk/JLanguageTool/src/java/org/languagetool/gui/ResultArea.java
2012-07-08 12:07:40 UTC (rev 7678)
@@ -219,20 +219,26 @@
}
private void handleRuleLinkClick(String uri) throws IOException {
- final RuleLink ruleLink = RuleLink.getFromString(uri);
- final String ruleId = ruleLink.getId();
- final Set<String> disabledRuleIds = config.getDisabledRuleIds();
- if (uri.startsWith(DEACTIVATE_URL)) {
- disabledRuleIds.add(ruleId);
- languageTool.disableRule(ruleId);
- } else {
- disabledRuleIds.remove(ruleId);
- languageTool.enableRule(ruleId);
+ final Cursor prevCursor = getCursor();
+ setCursor(new Cursor(Cursor.WAIT_CURSOR));
+ try {
+ final RuleLink ruleLink = RuleLink.getFromString(uri);
+ final String ruleId = ruleLink.getId();
+ final Set<String> disabledRuleIds = config.getDisabledRuleIds();
+ if (uri.startsWith(DEACTIVATE_URL)) {
+ disabledRuleIds.add(ruleId);
+ languageTool.disableRule(ruleId);
+ } else {
+ disabledRuleIds.remove(ruleId);
+ languageTool.enableRule(ruleId);
+ }
+ config.setDisabledRuleIds(disabledRuleIds);
+ config.saveConfiguration(languageTool.getLanguage());
+ allRuleMatches = languageTool.check(textArea.getText());
+ reDisplayRuleMatches();
+ } finally {
+ setCursor(prevCursor);
}
- config.setDisabledRuleIds(disabledRuleIds);
- config.saveConfiguration(languageTool.getLanguage());
- allRuleMatches = languageTool.check(textArea.getText());
- reDisplayRuleMatches();
}
private void reDisplayRuleMatches() {
This was sent by the SourceForge.net collaborative development platform, the
world's largest Open Source development site.
------------------------------------------------------------------------------
Live Security Virtual Conference
Exclusive live event will cover all the ways today's security and
threat landscape has changed and how IT managers can respond. Discussions
will include endpoint security, mobile security and the latest in malware
threats. http://www.accelacomm.com/jaw/sfrnl04242012/114/50122263/
_______________________________________________
Languagetool-cvs mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/languagetool-cvs
