Discourse/discourse-placeholder-theme-component
1
0
Fork 0
mirror of https://github.com/discourse/discourse-placeholder-theme-component.git synced 2026-10-28 00:08:18 -04:00
Code Issues Packages Projects Releases Wiki Activity
68 Commits 16 Branches 0 Tags
b4ffb4772f60feab9a177dfa04a49cf0d067ce44
Commit Graph
2 Commits
Author SHA1 Message Date
David Taylor a37e4bf266 FIX: Restore replacement functionality for a[href] attributes (#42) 
Followup to 948634fe31
 2024-09-04 09:31:17 +01:00
David TaylorandJoffrey JAFFEUX 569b566d38 FIX: Multiple placeholders in a single post (#40) 
The changes in 948634fe31 meant that only the most-recently-changed placeholder is actually applied.

This commit refactors things so that we store all placeholder values in JS, and then apply them all in a single pass over the DOM. As well as fixing the bug, this should be a significant perf improvement for posts with lots of placeholders

Also introduces some simple system specs.
---------

Co-authored-by: Joffrey JAFFEUX <[email protected]>
 2024-08-29 12:37:21 +01:00