Uploaded image for project: 'Camunda Optimize'
  1. Camunda Optimize
  2. OPT-1161

"to" label in start date filter is too low

XMLWordPrintable

    • Icon: Bug Report Bug Report
    • Resolution: Fixed
    • Icon: L3 - Default L3 - Default
    • 2.0.0
    • None
    • frontend
    • None

      Reproduce:

      • open the "Add Start Date Filter"

      Expected:

      • "to" label between start and end date is at same height as the date input values

      Observed:

      • "to" label is too low (see screenshow)

        This is the controller panel for Smart Panels app

          1. screenshot-1.png
            screenshot-1.png
            31 kB

            [OPT-1161] "to" label in start date filter is too low

            Franz added a comment -

            Cannot reproduce the issue on my machine (MacOS 10.12.6, Chrome Version 65.0.3325.162). Even if I could, a user would be hard pressed to notice the issue nor would it be any serious degradation of the user experience.

            Franz added a comment - Cannot reproduce the issue on my machine (MacOS 10.12.6, Chrome Version 65.0.3325.162). Even if I could, a user would be hard pressed to notice the issue nor would it be any serious degradation of the user experience.

            Sebastian Stamm added a comment -

            I don't think we should use an input element just to get the styling right. A span element would be semantically more correct. Using an input element also messes up the tab order: Pressing tab in the first date input previously correctly selected the second date input. Now it focusses the to label.

            Sebastian Stamm added a comment - I don't think we should use an input element just to get the styling right. A span element would be semantically more correct. Using an input element also messes up the tab order: Pressing tab in the first date input previously correctly selected the second date input. Now it focusses the to label.

            Sebastian Stamm added a comment -

            I could not find any new commit related to this issue

            Sebastian Stamm added a comment - I could not find any new commit related to this issue

            Johannes added a comment -

            Try again

            Johannes added a comment - Try again

            Sebastian Stamm added a comment -

            Much better

            Sebastian Stamm added a comment - Much better

              Unassigned Unassigned
              sebastian.stamm Sebastian Stamm
              Votes:
              0 Vote for this issue
              Watchers:
              3 Start watching this issue

                Created:
                Updated:
                Resolved: