Make editor not lose focus when you click a toolbar button (or think it loses focus)

RESOLVED INVALID

Status

P2
normal
RESOLVED INVALID
10 years ago
8 years ago

People

(Reporter: ben, Unassigned)

Tracking

Details

(Reporter)

Description

10 years ago
From Jesse
This is a mass migration from Mozilla Labs :: Bespin to Bespin :: General.

This bug likely still needs to be triaged and categorized.
Component: Bespin → General
Product: Mozilla Labs → Bespin
QA Contact: bespin → general
Target Milestone: -- → ---
(Reporter)

Comment 2

10 years ago
Pretty silly that we haven't fixed this yet
Priority: -- → P2

Comment 3

10 years ago
The editor triggers onblur actions when something else is clicked, it seems to make the most sense to add explicit re-focus events JUST for the toolbar buttons. This patch should fix save / preview / fontsize, does the same fix need to be applied anywhere else?

diff -r 1dfcd89b80f0 frontend/js/bespin/editor/toolbar.js
--- a/frontend/js/bespin/editor/toolbar.js      Thu Aug 20 16:25:16 2009 +0100
+++ b/frontend/js/bespin/editor/toolbar.js      Fri Aug 21 08:43:15 2009 -0700
@@ -143,6 +143,7 @@
 
             dojo.connect(save, 'click', function() {
                 bespin.get("editor").saveFile();
+                bespin.get("editor").setFocus(true);
             });
         },
 
@@ -206,6 +207,7 @@
             dojo.connect(preview, 'click', function() {
                 // Defaults to current
                 bespin.preview.show();
+                bespin.get("editor").setFocus(true);
             });
         },
 
@@ -232,8 +234,9 @@
                 dojo.connect(fontsize, 'click', function() {
                     currentFontSize = (currentFontSize > 2) ? 1 : currentFontSize + 1;
                     bespin.publish("settings:set:fontsize", [{ value: fontSizes[currentFontSize] }]);
+                    bespin.get("editor").setFocus(true);
                 });
             })();
         }
     }
-});
\ No newline at end of file
+});
(Reporter)

Comment 4

9 years ago
setting to P2 and targetting 0.4.3 (because ryan went to trouble to patch, we should process if we can)
Target Milestone: --- → 0.4.3
(Reporter)

Updated

9 years ago
Target Milestone: 0.4.3 → 0.4.4

Updated

9 years ago
Target Milestone: 0.4.4 → ---

Comment 5

8 years ago
ACETRANSITION

The Skywriter project has merged with Ajax.org's Ace project (the full server part of which is their Cloud9 IDE project). Background on the change is here:

http://mozillalabs.com/skywriter/2011/01/18/mozilla-skywriter-has-been-merged-into-ace/

The bugs in the Skywriter product are not necessarily relevant for Ace and quite a bit of code has changed. For that reason, I'm closing all of these bugs. Problems that you have with Ace should be filed in the Ace issue tracker at GitHub:

https://github.com/ajaxorg/ace/issues
Status: NEW → RESOLVED
Last Resolved: 8 years ago
Resolution: --- → INVALID
You need to log in before you can comment on or make changes to this bug.