Error font size is too small
Bug #711124 reported by
Tim Penhey
This bug affects 1 person
| Affects | Status | Importance | Assigned to | Milestone | |
|---|---|---|---|---|---|
| LAZR Javascript Library |
Fix Released
|
Medium
|
Tim Penhey | ||
| Launchpad itself |
Fix Released
|
High
|
Tim Penhey | ||
Bug Description
The error font size should not be 0.5em, but it should be up to the call sites to make the error text smaller when editing larger text.
Related branches
lp:~thumper/lazr-js/multi-line-editor-goodness
- Deryck Hodge (community): Approve (code)
-
Diff: 432 lines (+188/-52)4 files modifiedexamples/inlineeditor/index.html (+29/-4)
src-js/lazrjs/inlineedit/assets/skins/sam/editor-skin.css (+13/-5)
src-js/lazrjs/inlineedit/editor.js (+44/-29)
src-js/lazrjs/inlineedit/tests/inline_edit.js (+102/-14)
Revision history for this message
| Tim Penhey (thumper) wrote | #1 |
| Changed in launchpad: | |
| status: | New → Triaged |
| importance: | Undecided → High |
| tags: | added: recipe |
| Changed in launchpad: | |
| assignee: | nobody → Tim Penhey (thumper) |
| Changed in launchpad: | |
| status: | Triaged → In Progress |
Revision history for this message
| Launchpad QA Bot (lpqabot) wrote | #2 |
| tags: | added: qa-needstesting |
| Changed in lazr-js: | |
| status: | In Progress → Fix Committed |
| Changed in launchpad: | |
| status: | In Progress → Fix Released |
| Changed in lazr-js: | |
| status: | Fix Committed → Fix Released |
| tags: |
added: qa-ok removed: qa-needstesting |
To post a comment you must log in.
Adding a launchpad task to make sure we change the styles of the inline editors for the H1 tags.