You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Current Vrembem JavaScript components are written in a single file. This can make it hard to understand their features, test and keep organized.
Possible Solution
Similar to the recent modular update on scroll-stash, should break up the index file to feature modules with a consistent structure for core and settings modules.
General Structure
Here's a concept for a general component structure:
core.js - Consists of the core definition of the component and the API that gets exposed for distribution. Every component would attach to it's API settings, init and destroy and check autoInit here.
settings.js - Contains the default options object.
<feature>.js - The rest of the module exports would be grouped by feature. For example, if the component has state management, that would be included via a state module defined in src/state.js.
JavaScropt Components
A list of JS components that would need this update:
Drawer
Modal
Checkbox
The text was updated successfully, but these errors were encountered:
Problem
Current Vrembem JavaScript components are written in a single file. This can make it hard to understand their features, test and keep organized.
Possible Solution
Similar to the recent modular update on scroll-stash, should break up the index file to feature modules with a consistent structure for core and settings modules.
General Structure
Here's a concept for a general component structure:
core.js
- Consists of the core definition of the component and the API that gets exposed for distribution. Every component would attach to it's APIsettings
,init
anddestroy
and checkautoInit
here.settings.js
- Contains the default options object.<feature>.js
- The rest of the module exports would be grouped by feature. For example, if the component has state management, that would be included via astate
module defined insrc/state.js
.JavaScropt Components
A list of JS components that would need this update:
The text was updated successfully, but these errors were encountered: