Skip to content

Move drawers usage of .content-wrapper for height offset to constant

The following discussion from !121618 (merged) should be addressed:

  • @mrincon started a discussion: (+1 comment)

    Question (for follow-up): Is .content-wrapper a general solution we should use for most (all?) drawers?

    If so we could move it next to DRAWER_Z_INDEX at app/assets/javascripts/lib/utils/constants.js so folks know it's the right one to use. DRAWER_CONTAINER_CLASS already exists in other places.

Update usage of .content-wapper in getContentWrapperHeight(.content-wrapper) to use a global constant like DRAWER_Z_INDEX e.g DRAWER_CONTAINER_CLASS. There is already some usage of localised DRAWER_CONTAINER_CLASS constant which should also be updated.