-
-
Notifications
You must be signed in to change notification settings - Fork 3.7k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[4.0] Redo the com_templates #32896
The head ref may contain hidden characters: "\u2014com-templates\u20144.0-dev"
[4.0] Redo the com_templates #32896
Conversation
administrator/components/com_templates/src/Model/StylesModel.php
Outdated
Show resolved
Hide resolved
looks good. will test properly later |
administrator/components/com_templates/tmpl/styles/default_modal_install.php
Outdated
Show resolved
Hide resolved
Go to Home Dashboard. |
@Quy done, thanks, good catch |
I dont know, should I create new issue, but I write my suggestion in this PR. When I upgrade Joomla 4.0.0 beta8-dev to new daily version (check updates and reinstall Joomla core files), sometimes there is a notification "%number% overriden files changed" and in this frame text changed to "%number% overrides to check". I go to administrator/index.php?option=com_templates&view=templates, in override files column show yellow "changes found". Usually it happens with helixultimate template (new 2.0.0 beta version). With cassiopeia that notification not showing. My suggestion: show, which of files has changed or overriden. Maybe it be useful for @dgrammatiko attempt to rewrite com_template. Or it's unnecessary function? |
And if there are 25 changed files? |
That was easy
This needs some query tweaking |
@brianteeman I don't know about that, it's just my suggestion :) I think, when number of changed files are more than 15-20, it's hard to read list all of them. Maybe it's unnesessary function. |
@dgrammatiko I think you also need to rethink the actions menu - it might not even be needed - so that its clear if the action is for the style or the template |
Looks good. The Title bar might be Templates: Site or Templates Administrator as required. I think the default template should no be 'deletable'. And I guess the double icons for Info and Preview will get fixed. The first screen grab shows a different side menu order - I assume that is a user choice and not an intended default (no need to answer that unless it is). This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/32896. |
Why don't you use the exact UX that we proposed? |
@kawshar I think I kept it almost intact (apart from changing some elements for a11y and using the existing styling, not going to restyle everything here) |
Another thought: I wonder if This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/32896. |
@ceford that does indeed make sense when you only have one style. As soon as you have multiple styles it becomes more confusing. |
?> | ||
<style> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
do you really need this?
@dgrammatiko let me know if I can help test the feature! 😉 |
- Custom names only for components,modules (no layouts, plugins as there is no way to redeclare a different layout name for them) - Wire the backend for the creation of the file, WIP
@RickR2H it's not fully functional atm (it creates just the html dir structure in the template) but if you have some time and test the front end and give some feedback that would be valuable |
It was my dream since I am using Joomla to have the two template views in one, thanks for working on that. But I have a few remarks and think ithat the appearance should be consistent with atum.
|
Generally speaking, this is still WIP so things will eventually be more consistent
There are 2 commands there: send email to support and get the template description
Legacy is referring to the templates that CANNOT have child templates, Modern will be the ones that do support them
The card footer is not really representative of what will be there finally, it needs work
There's a star icon that is supposed to do that, the logic is missing atm About all the icons, I just picked random ones but it's an easy task to change them |
This one could be revisited in v5 |
@dgrammatiko please re-open this. Dont allow yourself to be bullied by ignorant fools (unless of course they were correct and you were paid to mess joomla up as claimed on tiwtter) |
@brianteeman this is replaced by
for 4.1 according to the maintainers' guidance. This can still be valid for another release. |
I know this thread is closed. But I would like to know whether such a view will be implemented in the next versions? |
@rinka88 I can only say that I'm not working on this anymore (and nobody asked me to do so), for anything related to v5 you have to ask @HLeithner |
it would be a pity if such good things were lost. And you can not offer them in the form of pr? |
I'm afraid this is not the right way, let me explain: finishing this PR means I have to personally spend many man-hours coding/styling/testing/etc with a huge uncertainty if the PR will ever get merged (if not it was just wasted time). I cannot afford to dedicate the needed time right now, but even I was able to do so, I wouldn't do that before the maintainers gave me a clear signal that this work won't be ignored/turned down... Anyways, September is a very busy month for me and I think after September there's no reasonable time to make this happen. Maybe in 5.1 or later (if people are not too attached to b/c breaks within a single component) and there is a competent release leader... |
Pull Request for Issue # .
Summary of Changes
One view to rule them all.
This PR largely is the adoption of the work that was done by @kawshar and his team in their backend template. So all the kudos should go to them, I'm just bringing their code to the current branch (with their permissions)
Testing Instructions
This is part of a wider rewrite of com_templates, probably has some a11y issues but should be functionally on par with the previous state (actually should be better as everything is one click away)
Actual result BEFORE applying this Pull Request
Expected result AFTER applying this Pull Request
Install template with one click
Documentation Changes Required
@brianteeman can you review the a11y and the strings here?