Skip to content

Commit

Permalink
Issue #2251: Corrected html structure of customer date(time) dynamic …
Browse files Browse the repository at this point in the history
…fields to fit in with new rendering.

Also adjusted js and css label code accordingly.
  • Loading branch information
stefanhaerter authored and svenoe committed Sep 18, 2023
1 parent 3d477ff commit 4d4f8dc
Show file tree
Hide file tree
Showing 4 changed files with 9 additions and 5 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -14,6 +14,7 @@
# along with this program. If not, see <https://www.gnu.org/licenses/>.
# --

<div class="DynamicFieldDate">
[% Data.DateSelectionHTML %]
[% IF Data.Mandatory %]
<div id="[% Data.DivIDMandatory %]" class="TooltipErrorMessage">
Expand All @@ -29,3 +30,4 @@
</p>
</div>
[% END %]
</div>
Original file line number Diff line number Diff line change
Expand Up @@ -14,6 +14,7 @@
# along with this program. If not, see <https://www.gnu.org/licenses/>.
# --

<div class="DynamicFieldDate">
[% Data.DateSelectionHTML %]
[% IF Data.Mandatory %]
<div id="[% Data.DivIDMandatory %]" class="TooltipErrorMessage">
Expand All @@ -29,3 +30,4 @@
</p>
</div>
[% END %]
</div>
2 changes: 1 addition & 1 deletion var/httpd/htdocs/js/Core.UI.Datepicker.js
Original file line number Diff line number Diff line change
Expand Up @@ -170,7 +170,7 @@ Core.UI.Datepicker = (function (TargetNS) {
Container.wrapInner("<div class='oooDate'></div>");
Container.addClass('oooDateContainer');

var Label = $('label', Container.parent());
var Label = $('label', Container.parent().parent());
var Checkbox = $('.oooDate > input[type=checkbox]', Container);
var Icon = $("<i class='oooAltCheck ooofo'></i>").on('click', function() {
Checkbox.click();
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -390,10 +390,10 @@ label.oooFocus.LabelError {
}*/

/* Checkboxes & Date */
.FieldCell > div.Field.oooDateContainer,
.FieldCell > div.Field.oooCheckboxContainer,
.Row > div.Field.oooDateContainer,
.Row > div.Field.oooCheckboxContainer {
.FieldCell > div.Field > div.oooDateContainer,
.FieldCell > div.Field > div.oooCheckboxContainer,
.Row > div.Field > div.oooDateContainer,
.Row > div.Field > div.oooCheckboxContainer {
height: 38px;
padding-top: 16px;
padding-left: 16px;
Expand Down

0 comments on commit 4d4f8dc

Please sign in to comment.