Skip to content

Conversation

@driusan
Copy link
Collaborator

@driusan driusan commented Oct 21, 2022

when the label is empty for some elements, it still includes a col-sm-3 element which offsets the element and no way to suppress it. Other elements do not include the 3 column space for the label when there is no label provided. This causes our elements to look weird and inconsistent when used in places that we just want the element (ie. in a flexbox row.)

This fixes the

  • DateElement
  • TimeElement
  • NumericElement
  • FileElement

So that there is no space left for the label if the label is not provided.

when the label is empty for some elements, it still includes
a col-sm-3 element which offsets the element and no way to
suppress it. Other elements do not include the 3 column space
for the label when there is no label provided. This causes
our elements to look weird and inconsistent when used in places
that we just want the element (ie. in a flexbox row.)

This fixes the
- DateElement
- TimeElement
- NumericElement
- FileElement

So that there is no space left for the label if the label is
not provided.
@driusan driusan added Area: UI PR or issue related to the user interface Category: Cleanup PR or issue introducing/requiring at least one clean-up operation labels Oct 21, 2022
@maltheism maltheism added the Passed manual tests PR has been successfully tested by at least one peer label Oct 25, 2022
@driusan driusan merged commit fe21f3a into aces:main Oct 25, 2022
@ridz1208 ridz1208 added this to the 25.0.0 milestone Mar 6, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Area: UI PR or issue related to the user interface Category: Cleanup PR or issue introducing/requiring at least one clean-up operation Passed manual tests PR has been successfully tested by at least one peer

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants