stimulus_reflex

stimulus_reflex copied to clipboard

# Type of PR Bug Fix and Enhancement ## Description Extract fragment logic into `StimulusReflex::HTML::Part` and introduce `StimulusReflex::HTML::Document` (for page morphs) and `StimulusReflex::HTML::Fragment` (for selector morphs) ## Why should this...

marcoroth

Refactor SR client package and introduce the Reflex class

1

# Type of PR Refactor ## Description This PR refactors the SR client package to use the new `Reflex` object - living in the global `reflexes` store - while still...

leastbad

SR 4 can safely shrink request and response payload sizes

1

With a major version bump on the horizon, we can take a hard look at our payload sizes. IMO, much of the information that we're passing is redundant. I'm going...

leastbad

Using middleware causes Rails actions to be invoked twice

1

# Bug Report ## Describe the bug When a middleware is used, Rails actions are invoked twice for every Reflex action. ## To Reproduce Add a middleware: ```ruby class DummyMiddleware...

shepmaster

Just found out that this line: https://github.com/stimulusreflex/stimulus_reflex/blob/165c2fae77b164e2f82306f4c692f97040b9c7fd/lib/tasks/stimulus_reflex/install.rake#L41 fails in a new `jsbundling-esbuild` setup because the following is typically present: ```js import controllers from './**/*_controller.js' controllers.forEach(controller => { application.register(controller.name, controller.module.default) })...

julianrubisch

Toolset mnemonic workspace

1

### What is this pull request? The toolset that is most often used for working with stimulus reflex enhances vanilla rails to a degree that warrants separation by name. The...

Anakin100100

For example, CR's utils.js already has a debounce and xpathToElement function. I think there's an opportunity here to save on bundle size. Cc @marcoroth @ParamagicDev

julianrubisch

Module not found: Error: Can't resolve 'stimulus'

7

I've this problem when I run ```bundle add stimulus_reflex``` and then, showing ```Can't locate the stimulus_reflex NPM package. Either add it to your package.json as a dependency or use "yarn...

jusondac

2 Issues: Nonexistent Cable Ready operation selectors / data-reflex shorthand

1

# Bug Report ## Describe the bug Found 2 bugs while pairing with @marcoroth 1. If a reflex contains multiple CableReady operations and *some* of the operations have element selectors...

pooriajr

Do not import controllers in installer when already present

1

# Bug Report ## Describe the bug The installer might add an additional `import "controllers"` in `app/javascript/packs/application.js` when one is already present. ## Expected behavior There should be no duplicate...

julianrubisch