Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

expose bc_dynamic_js to javascript so we can always load these files #3058

Merged
merged 2 commits into from
Sep 21, 2023

Conversation

johrstrom
Copy link
Contributor

Fixes #2950 so that the javascript file (with path_selector and other updates) will get loaded all the time. Only when bc_dynamic_js is enabled (the config now exposed through javascript) will the javascript file create event handlers for the same.

Comment on lines -719 to -729
// simple function to sanitize css query strings
function sanitizeQuery(item) {
return item.replaceAll('.', '\\.');
}


function optionForEvent(target) {
let simpleName = shortId(target['id']);
return mountainCaseWords(simpleName);
};

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

These removes (and the one above) aren't really a part of this pull request - I just noticed in my linter that they're unused.

Copy link
Contributor

@Oglopf Oglopf left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good.

@johrstrom johrstrom merged commit f9906e6 into master Sep 21, 2023
20 checks passed
@johrstrom johrstrom deleted the bc-js-refactor branch September 21, 2023 19:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

path_selector currently dependant on bc_dynamic_js
3 participants