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
[mod_articles_popular]Code style #10576
Conversation
<field name="relative_date" type="text" default="30" | ||
<field | ||
name="relative_date" | ||
type="text" default="30" |
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.
move default down?
I have tested this item ✅ successfully on 40b8104 This comment was created with the J!Tracker Application at issues.joomla.org/joomla-cms/10576. |
<fieldset | ||
name="advanced"> | ||
|
||
<fieldsetname="advanced"> |
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.
missing space between fieldset and name ...
I have tested this item 🔴 unsuccessfully on 40b8104 This comment was created with the J!Tracker Application at issues.joomla.org/joomla-cms/10576. |
This PR has received new commits. CC: @andrepereiradasilva, @JoomliC This comment was created with the J!Tracker Application at issues.joomla.org/joomla-cms/10576. |
I have tested this item ✅ successfully on 2bad85a This comment was created with the J!Tracker Application at issues.joomla.org/joomla-cms/10576. |
1 similar comment
I have tested this item ✅ successfully on 2bad85a This comment was created with the J!Tracker Application at issues.joomla.org/joomla-cms/10576. |
RTC This comment was created with the J!Tracker Application at issues.joomla.org/joomla-cms/10576. |
Summary of Changes
Code style changes for mod_articles_popular
Testing Instructions
code review
cc @JoomliC @andrepereiradasilva