Opened 13 years ago
Closed 13 years ago
#4971 closed defect (fixed)
toolbar with dropdowns (editor font choices) broken on IE
Reported by: | Adam Peller | Owned by: | Adam Peller |
---|---|---|---|
Priority: | high | Milestone: | 1.1 |
Component: | Dijit | Version: | 0.9 |
Keywords: | Cc: | ||
Blocked By: | Blocking: |
Description
because they're not popups, the focus is not managed correctly. Fix for 1.0 or remove support for this feature until this is fixed.
Change History (15)
comment:1 Changed 13 years ago by
comment:2 Changed 13 years ago by
comment:3 Changed 13 years ago by
Milestone: | 1.0 → 1.1 |
---|---|
Owner: | changed from Adam Peller to liucougar |
Cougar, can you take a look at this? I tried using the focus code as well as calling the editor's bookmark code directly to save/restore focus, but nothing worked. It appears that Editor may already be trying to do this on behalf of the plugin. So why is there a focus problem only on IE? It almost looks like a cross-domain issue trying to access the iframe?
comment:4 Changed 13 years ago by
Priority: | normal → high |
---|
comment:6 Changed 13 years ago by
(In [11884]) FontChoice? editor plugin should have been marked 'experimental'. Refs #4971, Refs #4171
comment:7 Changed 13 years ago by
(In [11885]) FontChoice? editor plugin should have been marked 'experimental'. Refs #4971, Refs #4171
comment:9 Changed 13 years ago by
Cc: | alex removed |
---|---|
Owner: | changed from liucougar to alex |
comment:10 Changed 13 years ago by
Status: | new → assigned |
---|
comment:11 Changed 13 years ago by
comment:12 Changed 13 years ago by
Owner: | changed from alex to Adam Peller |
---|---|
Status: | assigned → new |
comment:14 Changed 13 years ago by
comment:15 Changed 13 years ago by
Resolution: | → fixed |
---|---|
Status: | new → closed |
There is an infrastructure for this, but it's complicated because you have to restore focus and selection *before* executing the font-change. Look at Menu.js for a related example. I think it would work something like: