better-html

better-html copied to clipboard

Replace "ast" gem with custom implementation

3

The "[ast](https://github.com/whitequark/ast/commits/master)" gem, a core dependency of better-html, is no longer actively maintained, with its most recent [non-trivial commit](https://github.com/whitequark/ast/commit/0b02fcf49659d7aada2d5246c2f84e9d2cedaef8) happening nearly 3 years ago. Having crucial functionality of the gem...

Forthoney

Partial incompatibility with rails 7.1

2

I realised after upgrading to rails 7.1 that when I encounter an exception in the view, I do not get the familiar `ActionDispatch::DebugExceptions` page anymore, but rather my `exceptions_app` After...

Intrepidd

So I'm aware that you can do `html_attributes("data-value": nil) if condition?` to return `"data-value"` if the condition is truthy. Is that the cleanest / only format? On reading the code,...

phantomwhale

Better HTML breaks Rails error page when web-console gem is used

2

Hey folks 👋 When @clayton-shopify hooked us up with better_html in https://github.com/Shopify/help/pull/7582, it introduced a bug that was fixed in https://github.com/Shopify/help/pull/7839. What ended up happening was that when the `config.allow_single_quoted_attributes...

jonniesweb

This is stemming from an issue running erb-lint on files with AlpineJS syntax (https://github.com/Shopify/erb-lint/issues/221), but it would be nice to add support to configure the parser to use some of...

clinejj

Readme .yml config update

1

Update the readme to use `YAML.unsafe_load` to parse regex values, `load` won't by default. I had the same issue as #106 , and this solved it. See [this psych discussion](https://github.com/ruby/psych/issues/489#issuecomment-841354484)....

thejonroberts

In the README, the documentation has this: > * `template_exclusion_filter`: This is called when determining whether to apply runtime checks on a `.erb` template. > When this `Proc` returns false,...

neanias

Add assertion to README.md testing block.

2

Rails 7.2 introduced warning for assertion-free tests. Good to recommend block with assertion (similar to rspec one) to prevent warnings.

simi

I stumbled across the following html snippet which crashes the parser: `bar`. The interpreted html from browsers (both chrome and firefox) is `bar` so the generated ast with the fix...

Earlopain

Bumps [rack](https://github.com/rack/rack) from 3.1.8 to 3.1.12. Changelog Sourced from rack's changelog. [3.1.12] - 2025-03-11 Security CVE-2025-27610 Local file inclusion in Rack::Static. [3.1.11] - 2025-03-04 Security CVE-2025-27111 Possible Log Injection in...

dependabot[bot]