| Summary: | StyledText needs to implement AccessibleEditableTextListener | ||
|---|---|---|---|
| Product: | [Eclipse Project] Platform | Reporter: | Carolyn MacLeod <carolynmacleod4> |
| Component: | SWT | Assignee: | Felipe Heidrich <eclipse.felipe> |
| Status: | RESOLVED FIXED | QA Contact: | |
| Severity: | normal | ||
| Priority: | P3 | CC: | Silenio_Quarti, skovatch |
| Version: | 3.7 | ||
| Target Milestone: | 3.7 M6 | ||
| Hardware: | PC | ||
| OS: | All | ||
| Whiteboard: | |||
| Bug Depends on: | |||
| Bug Blocks: | 324254 | ||
|
Description
Carolyn MacLeod
Regarding implementing setAttributes by default... at some point in the future, we could consider the possibility of having an "allow text style changes from accessibility tools" style bit (default false), so that applications that do not enforce text styles would get setAttributes behavior "for free". Would be nice if this got in before M5, because it is how outside applications will use the new AccessibleEditableText API. See the addAccessibleEditableTextListener code in the snippet in comment 0 for an example of how to implement. Fixed in HEAD Changed the implementation setTextAttributes() to do nothing, a few reason for that: 1) When StyledText has line event listener the setStyle API should not be used 2) The result of setTextAttributes could overwrite (or be overwritten) by user styles 3) The resources created in the implementation were not being freed. For the above, we agreed that setTextAttributes has to be implemented at the application level. |