Update: JSDoc documentation for drawer system (fixes #812)

[joe-replin]

Fix

• Fixes JSDocs - Drawer #812

Update

Added comprehensive JSDoc documentation to the drawer system (3 files):

• drawer.js - Singleton service documentation with complete API reference

  • File/module headers with architecture overview
  • Public events section listing all triggered events
  • All public API methods documented: isOpen, open(), openCustomView(), addItem(), hasItem(), close(), toggle(), remove()
  • Usage examples for menu item registration and custom view patterns
  • Known Issues section documenting DrawerCollection accessibility and missing update methods

• drawerView.js - Main controller view documentation

  • File/module headers explaining dual-mode operation (menu/custom view)
  • Configuration options and position logic documented
  • Public events section
  • Key methods documented: setDrawerPosition(), openCustomView(), showDrawer(), hideDrawer(), remove()
  • Known Issues section identifying single-item toggle workaround, force close hack, and tight coupling to navigation
  • Skipped Backbone config methods per JSDoc style guide

• drawerItemView.js - Menu item view documentation

  • File/module headers with lifecycle and model structure
  • Class documentation with usage notes
  • onDrawerItemClicked() method documented with callback pattern example
  • Noted internal creation pattern (not for direct instantiation)

All documentation follows the established JSDoc style guide patterns established in the notify system work.

Testing

1. JSDoc generates correctly for all three files
2. Cross-references ({@link}) resolve properly
3. Examples are syntactically valid
4. No JSDoc syntax errors or warnings

[oliverfoster]

Really good.

[oliverfoster]

The description and terminology aren't quite right throughout.

Replace 'menu items' with 'drawer items' and the word 'menu' for 'sidebar/drawer' as the word menu is used for menu plugins.
Remove 'navigation system' as the word navigation is used for a core module called 'navigation' which is the horizontal navigation bar, not the vertical sidebar.

[oliverfoster]

   * Adapt.trigger('navigation:toggleDrawer');

Consider as deprecated.

As a general principle for Adapt, any event triggered on a subject as if it were a function call that is not a triggered by the subject as notification of a state change should be replaced with a function API. Where a function alternative to an event trigger already exists, consider the event trigger to be deprecated.