Skip to content
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 ourterms of serviceandprivacy statement. We’ll occasionally send you account related emails.

Already on GitHub?Sign into your account

[5.1] Template Long Description #42651

Merged
merged 6 commits intofrom
Feb 3, 2024
Merged

Conversation

brianteeman
Copy link
Contributor

Pull Request for Issue identified with#42649.

Summary of Changes

Complete the code that truncates long descriptions.
As seen in
administrator/components/com_modules/tmpl/module/edit.php
administrator/components/com_plugins/tmpl/plugin/edit.php
administrator/components/com_scheduler/tmpl/task/edit.php
But only partially seen in
administrator/components/com_templates/tmpl/style/edit.php

each element that used "JGLOBAL_SHOW_FULL_DESCRIPTION", have a parent element with onclick attribute...

Testing Instructions

Update the descrption of the cassiopeia template inlanguage\en-GB\tpl_cassiopeia.iniso that it is over 500 characters long.

Actual result BEFORE applying this Pull Request

The entire description is displayed

Expected result AFTER applying this Pull Request

The description is truncated and a new tab is present with the full description

Link to documentations

Please select:

  • Documentation link for docs.joomla.org:

  • No documentation changes for docs.joomla.org needed

  • Pull Request link for manual.joomla.org: <link

  • No documentation changes for manual.joomla.org needed

Comment

Alternatively the partial support for this truncation functionality can be removed from com_templates

@sinahaghparast
Copy link

@brianteemanI saw changed file, thanks...

now it is better to choose one code style in these four file...

onclick="document.getElementById('myTab').activateTab(document.getElementById('description'));"
or
onclick="document.querySelector('[aria-controls=description]').click();"

@brianteeman
Copy link
Contributor Author

Anyone can submit a pull request

@toivo
Copy link
Contributor

toivo commentedJan 16, 2024

I have tested this item ✅ successfully on6501649

Tested successfully in Joomla 5.1.0-alpha3-dev of 15 January.


This comment was created with theJ!Tracker Applicationatissues.joomla.org/tracker/joomla-cms/42651.

@richard67
Copy link
Member

I've restored the previous human test result in the issue tracker as the 2 commits after the test were only a code style change and a clean branch update.

@Quy
Copy link
Contributor

Quy commentedJan 22, 2024

I have tested this item ✅ successfully on585dfd6


This comment was created with theJ!Tracker Applicationatissues.joomla.org/tracker/joomla-cms/42651.

@Quy QuyremovedtheFeaturelabelJan 22, 2024
@Quy
Copy link
Contributor

Quy commentedJan 22, 2024

RTC


This comment was created with theJ!Tracker Applicationatissues.joomla.org/tracker/joomla-cms/42651.

@joomla-cms-bot joomla-cms-botaddedtheRTC This Pull Request is Ready To CommitlabelJan 22, 2024
@Quy QuyaddedtheFeaturelabelJan 22, 2024
@LadySolveig LadySolveigadded this to theJoomla! 5.1.0milestoneFeb 2, 2024
@LadySolveig LadySolveigmerged commit5b315eainto joomla:5.1-dev Feb 3, 2024
3 checks passed
@joomla-cms-bot joomla-cms-botremovedtheRTC This Pull Request is Ready To CommitlabelFeb 3, 2024
@LadySolveig
Copy link
Contributor

Thank you@brianteemanand also vor testing@toivoand@Quy

@brianteeman
Copy link
Contributor Author

thanks

@brianteeman brianteemandeleted the satnight branch February 3, 2024 08:58
Sign up for free to join this conversation on GitHub.Already have an account?Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

7 participants