Skip to content

Refactor popups documentation into separate pages - #67

Merged
oblomov-dev merged 2 commits into
mainfrom
claude/add-popup-popover-submenu-nLzgW
May 20, 2026
Merged

Refactor popups documentation into separate pages#67
oblomov-dev merged 2 commits into
mainfrom
claude/add-popup-popover-submenu-nLzgW

Conversation

@oblomov-dev

Copy link
Copy Markdown
Member

Summary

Reorganized the popups documentation by splitting the monolithic popups.md file into separate, focused pages for better maintainability and navigation.

Key Changes

  • Renamed docs/development/popups/popups.md to focus exclusively on popup functionality

    • Removed popover and built-in popup sections
    • Updated title from "Popups, Popovers" to "Popups"
    • Converted section headers from ### to ## for better hierarchy
  • Created docs/development/popups/popover.md with dedicated popover documentation

    • Extracted popover example and usage instructions
  • Created docs/development/popups/built_in.md with pre-built popup classes reference

    • Extracted list of built-in popup classes with contribution note
  • Updated docs/.vitepress/config.mjs navigation structure

    • Changed "Popups, Popovers" link to point to /development/popups/popups
    • Added collapsible submenu with three items: Popups, Popover, and Built-In
    • Improved discoverability of related documentation

Benefits

  • Clearer separation of concerns between popups and popovers
  • Easier to locate and maintain specific documentation sections
  • Better navigation structure with collapsible menu in sidebar
  • Reduced cognitive load when reading focused documentation pages

https://claude.ai/code/session_01UN9ZiWFo9RujEUU88fK1WV

claude added 2 commits May 20, 2026 12:46
Replace the single popups.md with a submenu containing dedicated pages
for Popups, Popover and Built-In popups so each topic gets its own
entry in the sidebar.
Convert plural menu labels to singular for consistency across the
sidebar and nav: Sample App, Use Case, Event, Message, Error, Popup,
Browser Feature, Map, Expert Topic, Lock, UI5 Version, Advanced Topic,
User Exit, Smart Control, Technical Insight, Result, Tool, Resource,
Add-on, Reference, Release.

Also rename docs/development/popups/popups.md to popup.md and update
its H1 to match.
@oblomov-dev
oblomov-dev merged commit 5ba0a1b into main May 20, 2026
@oblomov-dev
oblomov-dev deleted the claude/add-popup-popover-submenu-nLzgW branch May 20, 2026 12:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants