Rewrite scrollup to use Vanila JS

Comment 7 months ago →
🇦🇺Australia VladimirAus Brisbane, Australia
Latest patch no longer apply.
Merge request !7Resolve #3325272 "Rewrite scrollup to" → (Merged) created by VladimirAus
Status changed to Needs review 7 months ago2:24pm 21 May 2024
Comment 7 months ago →
🇦🇺Australia VladimirAus Brisbane, Australia
Status changed to Needs work 7 months ago3:22am 22 May 2024
Comment 7 months ago →
🇦🇺Australia jannakha Brisbane!
- set default value for scroll_title in case it's empty
- title should not be hardcoded: `title="scroll_title"`
- you can use Template literals, `title=${scroll_title}`
- use semantic HTML, eg ... (not
...

)
Status changed to Needs review 7 months ago3:26am 22 May 2024
Comment 7 months ago →
🇦🇺Australia VladimirAus Brisbane, Australia
Thank you for review! 🧁
Updated.
Status changed to RTBC 7 months ago3:45am 22 May 2024
Comment 7 months ago →
🇦🇺Australia jannakha Brisbane!
this MR is good to go.
Pipeline finished with Skipped
7 months ago
#178770

VladimirAus → committed 6ff48c92 on 3.0.x

Issue #3325272 by VladimirAus, vknikolov, jannakha: Rewrite scrollup to...

Status changed to Fixed 7 months ago3:56am 22 May 2024
Comment 7 months ago →
🇦🇺Australia VladimirAus Brisbane, Australia
Thank you! Committed! 🥂
Comment 7 months ago →
System Message
Automatically closed - issue fixed for 2 weeks with no activity.

Merge Requests