Patch #40137
closedJstoolbar help files should import images from the asset pipeline
0%
Description
jstoolbar help files are pure html static files that are importing the images using html image elements, for example:
<img src="../../images/jstoolbar/bt_img.png" style="border: 1px solid #bbb;" alt="Image" />
We need to update these files in order to use the images generated by the asset pipeline. I see 2 options:
1. move all these images as CSS background images
2. refactor all help files in order to use a template that is rendered on server side. If we do this, it will make also easier to modify that files because you don't need to make the same change in all help files.
Files
Related issues
Updated by Marius BĂLTEANU 7 months ago
- Related to Feature #39111: Enable Asset Pipeline Integration using Propshaft added
Updated by Marius BĂLTEANU 6 months ago
- File Archive.zip Archive.zip added
I've started working on this and I'm attaching an intermediary version of the patch series:
0001-Add-route-to-serve-all-wiki-syntax-help-files-from-v- Introduces
HelpController
and a new routehelp/wiki_syntax/(:detailed)
to serve help files - Moves all wiki syntax help files to
app/views/help
and CSS files toapp/assets/stylesheets
- Reorganize all the help files per text formatter (
markdown
,common_mark
andtextile
)
- Remove languages from wiki syntax help files that are identical with en language.
- Fall back to the english help page if there is none for the current language.
- Changes JS toolbar help button to use the new route
- Just add the erb extension to all files.
- Move inline style for img border to css file to cleanup a little bit the view files.
- Includes the css file from the asset pipeline.
- Update link that points to detailed version.
- Include images using
image_tag
. - Add tests for new route and
HelpController
.
I hope I will finish this until the end of this month.
Updated by Marius BĂLTEANU 6 months ago
- File 0008-Replace-static-image-tag-with-image_tag-in-order-to-.patch 0008-Replace-static-image-tag-with-image_tag-in-order-to-.patch added
- File 0009-Replace-static-links-to-detailed-common-mark-with-th.patch 0009-Replace-static-links-to-detailed-common-mark-with-th.patch added
Added last two patches that get rid of the static image tags and links.
Updated by Marius BĂLTEANU 6 months ago
I've committed all the patches, still need to add tests.
Updated by Marius BĂLTEANU 6 months ago
- Category set to Text formatting
- Status changed from New to Closed
- Assignee set to Marius BĂLTEANU
- Resolution set to Fixed
I think I've finished all the minimum changes in order to restore the old functionality.
Updated by Marius BĂLTEANU 6 months ago
- Tracker changed from Defect to Patch
- Resolution deleted (
Fixed)
Updated by Marius BĂLTEANU 4 months ago
- Related to Patch #40559: Fix incorrect icon image paths for Wiki help pages added