[MGNLUI-3296] Show remaining length on input fields Created: 15/Dec/14 Updated: 06/Aug/15 Resolved: 17/Apr/15 |
|
| Status: | Closed |
| Project: | Magnolia UI |
| Component/s: | dialogs, user interaction |
| Affects Version/s: | 5.3.5 |
| Fix Version/s: | 5.3.9 |
| Type: | Improvement | Priority: | Neutral |
| Reporter: | Stefan Jahn | Assignee: | Christoph Meier |
| Resolution: | Fixed | Votes: | 1 |
| Labels: | 5.3.9, support, ux | ||
| Remaining Estimate: | Not Specified | ||
| Time Spent: | Not Specified | ||
| Original Estimate: | Not Specified | ||
| Attachments: |
|
||||
| Issue Links: |
|
||||
| Template: |
|
||||
| Patch included: |
Yes
|
||||
| Acceptance criteria: |
Empty
|
||||
| Task DoD: |
[ ]*
Doc/release notes changes? Comment present?
[ ]*
Downstream builds green?
[ ]*
Solution information and context easily available?
[ ]*
Tests
[ ]*
FixVersion filled and not yet released
[ ] 
Architecture Decision Record (ADR)
|
||||
| Date of First Response: | |||||
| Description |
|
Use case: The recommended length is useful because sometimes you will not restrict the editor to a hard limit. We needed this recommendation so that the design is still looking good on mobile devices and the text doesn't broke it too much. This feature works with TextField and TextArea. I didn't try with a richTextEditor. I included the necessary java implementation files, 1 screenshot and here a small adjustment for css. .extended-textfield-label { text-align: right; color: #666666; white-space: nowrap; line-height: 25px; right: 2px; position: relative; font-size: 10px; top: -12px; height: 18px; } |
| Comments |
| Comment by Mikaël Geljić [ 19/Feb/15 ] |
|
Hi Stefan, This was indeed validated by UX group; we will simply make it a Vaadin Extension (not an extended field). This allows for nicer composition in the field factory, and has a lower component/layout foot-print. Thanks for the work! |
| Comment by Stefan Jahn [ 30/Mar/15 ] |
|
Hi, |
| Comment by Christoph Meier [ 02/Apr/15 ] |
|
As proposed above vaadin extension has been implemented (extending com.vaadin.server.AbstractExtension). |
| Comment by Christoph Meier [ 02/Apr/15 ] |
|
Commit is on branch " |
| Comment by Aleksandr Pchelintcev [ 15/Apr/15 ] |
|
Some things worth considering:
|
| Comment by Christoph Meier [ 17/Apr/15 ] |
|
Improvements added according to review. |