? ? Pending

User tests: Successful: Unsuccessful:

avatar dgrammatiko
dgrammatiko
23 Mar 2021

Pull Request for Issue # .

This is a retake (obviously a lot different) to my first PR ever for Joomla #4052 (wow 7 years...). It seems I did a bunch of PRs but never came around solving my own problem in a way that the core would enable Front End developers to customise the renderers without the need to roll their own plugin. Basically, this PR just completes the work that someone else started way back when introduced the message jLayout. In that sense is not a new feature, it just contributes to the goal that everything producing HTML code is overridable.

Summary of Changes

  • Move the code of the renderers to their respective JLayout files

Why?

Renderers produce fragments of HTML (or XML in case of RSS) but their internals are hard to be overridden (esp for people that are not familiar with the idea of Joomla's plugins). Moving them to jLayouts makes them immediately more approachable as in a template you can override them and plugin your own changes.

Testing Instructions

Test that all 3 templates working fine (front end, back end, installation)

Actual result BEFORE applying this Pull Request

Expected result AFTER applying this Pull Request

Documentation Changes Required

??? Are the jLayouts documented somewhere?

avatar dgrammatiko dgrammatiko - open - 23 Mar 2021
avatar dgrammatiko dgrammatiko - change - 23 Mar 2021
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 23 Mar 2021
Category Layout Libraries
avatar brianteeman
brianteeman - comment - 23 Mar 2021

It is way too late in the development cycle to introduce something so extensive - my 2c

avatar dgrammatiko
dgrammatiko - comment - 23 Mar 2021

It is way too late in the development cycle to introduce something so extensive

Can go to 4.1 or another release could introduce this. I just keep hacking my way around this problem for way too many years so I thought maybe I can fix this in a way that makes Joomla more approachable for front end devs (eg https://github.com/dgrammatiko/sloth/tree/step-6-inherritable/sloth/helper)

avatar brianteeman
brianteeman - comment - 23 Mar 2021

We've more important things to do right now - like making 4.0 usable for people who are not front end devs - which it currently is not

avatar dgrammatiko
dgrammatiko - comment - 23 Mar 2021

We've more important things to do right now

Sure, I'm also working on a couple of release blockers. This was a quick one as I was doing some side work on a project and realised that I still have to hack my way around to get the renderers to echo what I want. Also just because I opened a PR doesn't necessarily mean that it's gonna be accepted or merged (you know how things work here)...

avatar brianteeman
brianteeman - comment - 23 Mar 2021

It also distracts from the current issues - perhaps rename it as [4.1] ?

avatar dgrammatiko dgrammatiko - change - 23 Mar 2021
Title
[4.0] Expose renderers through JLayout
[4.1] Expose renderers through JLayout
avatar dgrammatiko dgrammatiko - edited - 23 Mar 2021
avatar dgrammatiko
dgrammatiko - comment - 23 Mar 2021

Done

avatar dgrammatiko dgrammatiko - change - 24 Mar 2021
Labels Added: ?
avatar dgrammatiko dgrammatiko - change - 5 May 2021
Status Pending Closed
Closed_Date 0000-00-00 00:00:00 2021-05-05 09:20:41
Closed_By dgrammatiko
Labels Added: ?
avatar dgrammatiko
dgrammatiko - comment - 5 May 2021

I guess plugins are working fine also here, so Frontenders using Joomla should be also PHP experts...

avatar dgrammatiko dgrammatiko - close - 5 May 2021

Add a Comment

Login with GitHub to post a comment