-
Notifications
You must be signed in to change notification settings - Fork 26
Define form input elements #735
Comments
Regarding the "Limited": What about bigger text input fields aka text areas? Do you want to incorporate them here, too. Or just 1-line-fields? For bigger text input, we usually show 3 lines and, if there's additional text, there'll be a scrollbar. Sometimes the input fields "grows" with the amount of text a user enters. We would need a defined rule how it should be. I prefer the latter one, because text is better readable without scrolling. |
Yes. Text areas have to be included. Text area rule on #735 (comment) sound good. We'll incorporate that. Case for #735 (comment) will be the Form Input with Action as per the cases enumerated on #733 (comment) Validation stuff will be addressed on #736 |
Form input main elements
|
@joseegm Did you have the chance to look at what we are putting together for form fields in Smart Edit/Personalization? It might be good to piggy back off of that since we've had many discussions and tests related to these? https://hybris.app.box.com/files/0/f/23235450874/1/f_163459049832 |
@joseegm I would watch out for the read only fields. Because of the layout, it was hard to not have them in the same position as without the field. In Smart Edit/Personalization, we are looking at a gray box behind it to make it look better on the page. This is another area where it helps to see the combination of the states of these fields in a single form rather than seeing them as separate entities. |
@jeannewalters Didn't Talked with Irla about that yet. But we will. Agree with the layout on Read Only input fields. We'll address that using the pertinent padding on the HTML/CSS. Also like the idea of having a visual element to keep them in place. Gonna try that. The toggles will be treated on a separate issue. States and validation are been treated on #736. Back to this issue in particular. |
Per Jose: If this is the case, should we wait to have a bigger conversation about leveraging what SmartEdit is already using before starting to recreate this? |
@LeoT7508 Still we need to finish #733, #736 and #738, before starting the visuals on #737. |
Every form element supports a From a programatic standpoint, it should be easier to output the same forms for read vs. edit mode and toggle only attribute. And the form fields maintain a tab order that a readonly DIV would not. Is there a reason we are using non-form elements.
In the case above, the |
Form input main elements
|
With the updated on #735 (comment) we cover all Elements for the Form Input field, with element names and the expected behaviour now. Details and Visuals and alignments for the Read Only Form Field will be addressed on #733 and #737 respectively. |
This is done. |
Goals:
Preliminary
The text was updated successfully, but these errors were encountered: