Toolbar Menu with Javascript disabled should be static

Created on 29 May 2020, over 4 years ago
Updated 3 April 2023, over 1 year ago

Pre-requisite-
1. Go to browser setting:
chrome://settings/content/javascript?search=content
2. Disable the javascript.

Steps to Replicate -
1. Go to the admin site.
2. Go to /admin/structure/views/view/content/edit (or any page with a vertical scroll)
3. Scroll the page , the toolbar menu gets moved downwards.
4. Go to /admin/structure/views
5. Scroll the page , the toolbar menu gets moved downwards.

Current Result -
If you disable javascript and visit the views ui and views page than on scrolling the page , the toolbar menu gets scrolled down , moved on scrolling.

Expected Result-
It should be static.

Refer to the attached video for more clearance.

🐛 Bug report
Status

Needs work

Version

10.1

Component
Toolbar 

Last updated 4 days ago

  • Maintained by
  • 🇫🇷France @nod_
Created by

🇮🇳India priyanka.sahni

Live updates comments and jobs are added and updated live.
Sign in to follow issues

Comments & Activities

Not all content is available!

It's likely this issue predates Contrib.social: some issue and comment data are missing.

  • The Needs Review Queue Bot tested this issue. It either no longer applies to Drupal core, or fails the Drupal core commit checks. Therefore, this issue status is now "Needs work".

    Apart from a re-roll or rebase, this issue may need more work to address feedback in the issue or MR comments. To progress an issue, incorporate this feedback as part of the process of updating the issue. This helps other contributors to know what is outstanding.

    Consult the Drupal Contributor Guide to find step-by-step guides for working with issues.

  • Status changed to Needs review over 1 year ago
  • Status changed to RTBC over 1 year ago
  • 🇺🇸United States smustgrave

    Confirmed the issue in the issue summary and the patch #42 resolved it. Attaching screenshots.

    Just FYI to help get the message out there.

    Starting March 2023, simple rerolls, rebases, or merges will no longer receive issue credit. Only rerolls that address a merge conflict will be credited, and the merge conflict that was resolved must be documented in the text of an issue comment.
    To receive credit for contributing to this issue, assist with other outstanding tasks or unaddressed feedback.
    See the issue credit guidelines for more information.

  • Status changed to Needs work over 1 year ago
  • 🇦🇺Australia larowlan 🇦🇺🏝.au GMT+10

    I think we need separate patches for D9 and 10 here per #33

  • 🇮🇳India ranjith_kumar_k_u Kerala

    Created a patch for D9 (from #42) and I can confirm that the patch is working fine on the 9.5 dev version

  • Status changed to Needs review over 1 year ago
  • Status changed to Needs work over 1 year ago
  • 🇺🇸United States smustgrave

    Just noticed in #42 this was added to claro also. Was it confirmed to be an issue there?

Production build 0.71.5 2024