This repository has been archived by the owner on Mar 5, 2020. It is now read-only.
Adding display of fieldLabel and fieldDescription to Touch UI inner fields #8
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Hello!
This request contains the following:
If possible, I would like to get this change, or something close to it, into a release soon. My current use case has instances of the Multicompositefield component with multiple text fields per item. It is difficult to know which field is which with no labels! Therefore the main goal of this pull request is to get that accomplished. The majority of the css changes are intended to support that addition. Moving to a less file extension on the css should let us clean up the css further at a later date. Right now it is pretty rough, but working.
Please note, I'm testing this in AEM 6.2 SP1. I confirmed the behavior in a normal Component Dialog (both as a modal and expanded to full screen), but I don't have a good example of a read only instance. If there are test cases available, I would appreciate getting a list of them to run before this is merged in.
Let me know what I need to do to move this forward!
Thank you!
-Edward