kayx23 opened a new pull request, #2150:
URL: https://github.com/apache/apisix-website/pull/2150

   ## Summary
   
   - keep the site-wide DocSearch dialog and backdrop above the sticky API 
reference toolbar
   - keep the version menu within a 320 px viewport
   - provide 40×40 px minimum targets for the back link, version control, and 
compact download action
   - add browser regression coverage for Admin API and Control API in English 
and Chinese
   
   ## Why
   
   Docusaurus computes the site-wide DocSearch overlay at `z-index: 7`, while 
the shared API toolbar uses `z-index: 90` so it remains above Scalar sticky 
chrome. On Admin API and Control API pages, opening site search therefore 
leaves the toolbar undimmed and interactive above the backdrop, and the toolbar 
can cover the dialog.
   
   A literal replacement z-index is not sufficient because the production CSS 
optimizer rebases numeric layers. This PR uses an optimizer-safe 
custom-property value, matching the existing toolbar approach. The resulting 
DocSearch layer computes to `200`; Scalar search remains at `10000`.
   
   The wider responsive review also found that the fixed-width version menu 
leaves a 320 px viewport and that compact toolbar controls fall below a 40×40 
px hit area. The shared toolbar corrections cover both API references and both 
locales.
   
   ## Validation
   
   - reproduced the layering failure on production Admin API and Control API 
`next` pages in English and Chinese, desktop and mobile
   - synchronized multi-site production build: passed
   - clean bilingual Docusaurus production build: passed
   - generated CSS verification: DocSearch computes to `200`, toolbar to `90`
   - focused Playwright regression: 2 passed across desktop and mobile 
projects; each project checks English/Chinese Admin API and Control API at 320 
px
   - verified site search covers the toolbar, Scalar search remains topmost, 
version menus stay within the viewport, toolbar targets are at least 40×40 px, 
and Escape closes overlays
   - ESLint, stylelint pre-commit hook, and `git diff --check`: passed
   
   The isolated Docusaurus build continues to report the repository existing 
cross-site broken-link warnings; the full synchronized production build passes.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to