Skip to content

[TASK] Inline more Fluid viewhelpers in the .editorconfig - #2312

Merged
AnReiter merged 1 commit into
mainfrom
task/inline-more
Sep 8, 2026
Merged

[TASK] Inline more Fluid viewhelpers in the .editorconfig#2312
AnReiter merged 1 commit into
mainfrom
task/inline-more

Conversation

@oliverklee

Copy link
Copy Markdown
Contributor

Mark some Fluid viewhelpers that will result in inline HTML tags as inline.

@coveralls

coveralls commented Sep 7, 2026

Copy link
Copy Markdown

Coverage Report for CI Build 34237012663

Coverage remained the same at 100.0%

Details

  • Coverage remained the same as the base build.
  • Patch coverage: No coverable lines changed in this PR.
  • No coverage regressions found.

Uncovered Changes

No uncovered changes found.

Coverage Regressions

No coverage regressions found.


Coverage Stats

Coverage Status
Relevant Lines: 129
Covered Lines: 129
Line Coverage: 100.0%
Coverage Strength: 5.64 hits per line

💛 - Coveralls

@oliverklee oliverklee moved this from Todo to In Review in Best Practices code sprint Sep 7, 2026
@oliverklee
oliverklee force-pushed the task/inline-more branch 3 times, most recently from a078ad5 to 6a9f9fb Compare September 8, 2026 10:44
Comment thread .editorconfig Outdated
ij_html_do_not_indent_children_of_tags =
ij_html_enforce_quotes = false
ij_html_inline_tags = a,abbr,acronym,b,basefont,bdo,big,br,cite,cite,code,dfn,em,font,i,img,input,kbd,label,q,s,samp,select,small,span,strike,strong,sub,sup,textarea,tt,u,var,f:translate,f:format.number
ij_html_inline_tags = a,abbr,acronym,b,basefont,bdo,big,br,cite,cite,code,dfn,em,font,i,img,input,kbd,label,q,s,samp,select,small,span,strike,strong,sub,sup,textarea,tt,u,var,be:link.editRecord,be:link,be:moduleLink,be:link.editRecord,be:link.newRecord,f:ceil,f:count,f:first,f:floor,f:format.case,f:format.htmlspecialchars,f:format.number,f:format.printf,f:format.stripTags,f:format.raw,f:format.trim,f:format.urlencode,f:image,f:last,f:link.action,f:link.email,f:link.external,f:link.file,f:link.page,f:link.typolink,f:max,f:replace,f:round,f:spaceless,f:translate

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

cite & be:link.editRecord are double on this list

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Good catch!

@github-project-automation github-project-automation Bot moved this from In Review to In Progress in Best Practices code sprint Sep 8, 2026
@oliverklee
oliverklee force-pushed the task/inline-more branch 2 times, most recently from 2933c7f to 2d247de Compare September 8, 2026 12:36
@oliverklee
oliverklee requested a review from AnReiter September 8, 2026 12:36
@oliverklee oliverklee moved this from In Progress to In Review in Best Practices code sprint Sep 8, 2026
Mark some Fluid viewhelpers that will result in inline HTML
tags as inline.

Also sort the viewhelpers.
@AnReiter
AnReiter merged commit d0ff82d into main Sep 8, 2026
43 checks passed
@AnReiter
AnReiter deleted the task/inline-more branch September 8, 2026 14:57
@github-project-automation github-project-automation Bot moved this from In Review to Done in Best Practices code sprint Sep 8, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Development

Successfully merging this pull request may close these issues.

3 participants