Uploaded image for project: 'Nuxeo Platform'
  1. Nuxeo Platform
  2. NXP-27960

Fix double scrollbars on collapsible document page

    XMLWordPrintable

    Details

    • Type: Bug
    • Status: Resolved
    • Priority: Minor
    • Resolution: Fixed
    • Affects Version/s: 11.1-SNAPSHOT
    • Fix Version/s: 11.1, 2021.0
    • Component/s: Web UI
    • Release Notes Description:
      Hide
      • Removed --nuxeo-results-view-min-height (added in NXP-27652).
      • Deprecated --nuxeo-document-content-min-height in favor of --nuxeo-document-content-height (affects nuxeo-document-content).
      • Deprecated --nuxeo-document-trash-content-min-height in favor of --nuxeo-document-trash-content-height (affects nuxeo-document-trash-content).
      • The deprecated variables should be replaced by the new ones on themes and custom elements making use of them.
      Show
      Removed --nuxeo-results-view-min-height (added in NXP-27652 ). Deprecated --nuxeo-document-content-min-height in favor of --nuxeo-document-content-height (affects nuxeo-document-content ). Deprecated --nuxeo-document-trash-content-min-height in favor of --nuxeo-document-trash-content-height (affects nuxeo-document-trash-content ). The deprecated variables should be replaced by the new ones on themes and custom elements making use of them.
    • Tags:
    • Team:
      GANG
    • Sprint:
      nxGang Sprint 11.1.17
    • Story Points:
      2

      Description

      NXP-27652 introduced double vertical scrollbars on collapsible page. This is happens because we introduced css variables for both min-height and height on nuxeo-results, meaning that the min-height is overruled in some cases. This can be fixed by defining both css variables on nuxeo-document-content (and nuxeo-document-trash-content). But because this is reductant, we should refactor these variables. We can keep --nuxeo-results-view-height and remove --nuxeo-results-view-min-height. However, we do have other variables that used to rely on the deprecated variable but will now rely on the new variable, namely --nuxeo-document-content-min-height. Because the name is inconsistent, we should deprecate --nuxeo-document-content-min-height and instead name it --nuxeo-document-content-height, while still keeping the first as fallback, to keep compatibility. We should do the name for --nuxeo-document-trash-content-min-height.

        Attachments

          Issue Links

            Activity

              People

              • Votes:
                0 Vote for this issue
                Watchers:
                2 Start watching this issue

                Dates

                • Created:
                  Updated:
                  Resolved:

                  Time Tracking

                  Estimated:
                  Original Estimate - Not Specified
                  Not Specified
                  Remaining:
                  Remaining Estimate - 0 minutes
                  0m
                  Logged:
                  Time Spent - 4 hours, 30 minutes
                  4h 30m