Uploaded image for project: 'OpenShift Dev Console'
  1. OpenShift Dev Console
  2. ODC-6035

Kubevirt plugin should not change PatternFly typeahead styles globally

XMLWordPrintable

    • ODC Sprint 203
    • Low

      Description of problem:

      The monitoring custom time range dialog text fields are not sized correctly (doesn't have a similar height) IF the topology was opened earlier.

      It looks like that the affinity-expression-row.scss file from the kubevirt plugin is loaded in the topology which changes the PatternFly typeahead style globally.

      Prerequisites (if any, like setup, operators/versions):

      A cluster with monitoring

      Steps to Reproduce

      1. Test case 1:
        1. Open developer perspective
        2. Open Topology and switch to Monitoring
        3. Open time range dropdown and select "Custom time range"
      2. Test case 2:
        1. Open developer perspective
        2. Open Monitoring and reload your browser tab (to ensure other css rules are not loaded)
        3. Open time range dropdown and select "Custom time range"

      Actual results:

      Custom time range modal looks different in both test cases.

      Expected results:

      Custom time range modal should look similar on both cases and the text fields should have the same height.

      Reproducibility (Always/Intermittent/Only Once):

      Always

      Build Details:

      4.9

      Additional info:

      1. We should check if the .pf-m-typeahead rule is necessary. If yes we should ensure that it doesn't effect other components by adding a restrictive parent rule or something.
      2. Maybe it's possible to check if we have similar issues in our code?

        1. global-css.png
          108 kB
          Christoph Jerolimov
        2. custom-time-range-when-visiting-topology-first.png
          118 kB
          Christoph Jerolimov
        3. custom-time-range-after-reloading-on-monitoring-page.png
          125 kB
          Christoph Jerolimov

            viraj-1 Vikram Raj
            cjerolim Christoph Jerolimov
            Gajanan More Gajanan More (Inactive)
            Votes:
            0 Vote for this issue
            Watchers:
            2 Start watching this issue

              Created:
              Updated:
              Resolved: