Layout horizontal only works in some module positions. Is this intentional? Took me a while to work out why i couldnt get the columns to work

Layout horizontal only works in some module positions. Is this intentional? Took me a while to work out why i couldnt get the columns to work

No, no intentional at all... But the number of columns is related to the width of the container (maybe something for the description of the field). For example you can set 3 columns and in main-bottom you will get 2, because the container is smaller than 992px

The label "card style" confused me - I thought it referred to placing a border around the article. Not that it made the entire article a link,

When Card style is enable you should not be able to make the category a link as it will never work.

Same is true for the tags

Articles Info Layout

I think it makes more sense to describe this as single line and multiline as opposed to horizontal and vertical

I think it makes more sense to describe this as single line and multiline as opposed to horizontal and vertical

Good idea! Thank you

No matter what options you chose it is always output as a list. When you just have the article titles that makes sense but I dont believe it is correct to display articles with content in this way and it should be a div. We dont display a content blog view as a list

The label "card style" confused me - I thought it referred to placing a border around the article. Not that it made the entire article a link,

I understand card style as a combination of the two things. I was not sure, if it is good to put so much css on the module itself and better do it in Cassiopeia (as is now in the case of mod_articles_newsflash).

When Card style is enable you should not be able to make the category a link as it will never work.

Same is true for the tags

That is true ... Maybe it will be better to remove the possibility to display category and tags if card style is selected ... It is always difficult to get all ideas working well (and I have a lot of other ideas ;-) )

No matter what options you chose it is always output as a list. When you just have the article titles that makes sense but I dont believe it is correct to display articles with content in this way and it should be a div. We dont display a content blog view as a list

I was following a tutorial for accessible cards: https://gehirngerecht.digital/barrierefreie-cards-erstellen-schritt-fuer-schritt-anleitung/
They write: "A list is an advantage for screen readers because, among other things, it tells them how many elements to expect. "

They write: "A list is an advantage for screen readers because, among other things, it tells them how many elements to expect.

That's true but it would also be true for all the blog article views. For me it is an abuse of the list semantics

Show Introtext

I suggest making this default to Show. Otherwise it is very confusing when you change Titles Only to No and you still dont see the introtext

Articles Info Layout showon syntax is not correct

showon="title_only:0[AND]show_date:1[OR]show_category:1[OR]show_hits:1[OR]show_author:1"

It should not be displayed when title only = 1

I suggest some descriptions for params. There are several params

"..... filtering type" with selection inclusive / exclusive.

This is not self explaining, I have to check the code for unterstanding. Maybe due to language gap?

When we started with J4, we had the rule that labels on switches should be only yes/no or show/hide. There were only a few rare exceptions, for example the user state. In this module we have more exceptions. Woudn't it be better to make a selection dropdown if alternatives are other that yes/no or show/hide?

When we started with J4, we had the rule that labels on switches should be only yes/no or show/hide. There were only a few rare exceptions, for example the user state. In this module we have more exceptions. Woudn't it be better to make a selection dropdown if alternatives are other that yes/no or show/hide?

agreed

When we started with J4, we had the rule that labels on switches should be only yes/no or show/hide.

Oh, I didn't know (or I forgot it). I will change to select lists.

Oh, I didn't know (or I forgot it).

Never mind. There was no document or written rule, only discussions. Because with switches the user cannot see the possible options.
We could add this as "best practice rule" to the manual.

The article info data should be presented using the description element just as it is in the content component and not as a regular list element

The article info data should be presented using the description element just as it is in the content component and not as a regular list element

A description list needs <dt> and <dd> and at the moment we are only displaying the descripton but not the term.

• Option 1: always display the term
• Option 2: always make the term "visually hidden"
• Option 3: add a parameter show / hide term

well it should be the same as with an article

I like the choice linline / vertical for the info-data elements - and would like to have this also for articles.

Could we make that like in my PR, #43729
with a visually hidden <dl><dt> label and the inline / vertical display for <dd>?

Can you help me get up to speed? I don't yet know the status of this PR. What still needs to be done? What is missing? Where do you need help?

Looking at this right now, I would like to have this in the testing sampledata and if this is supposed to be the future, then we should directly replace all cases of the old modules in our sampledata plugins with this new plugin and ideally you shouldn't see the difference...

Can you help me get up to speed? I don't yet know the status of this PR. What still needs to be done? What is missing? Where do you need help?

Looking at this right now, I would like to have this in the testing sampledata and if this is supposed to be the future, then we should directly replace all cases of the old modules in our sampledata plugins with this new plugin and ideally you shouldn't see the difference...

this is supposed to exist in parallel with the other content modules - otherwise we are in migration hell

We've had these modules for at least 10 years, most likely even since Mambo times, so of course we wont be dropping them like hot potatoes. I don't see them being replaced anytime soon, so at least until the end of 8.x I would expect the old ones to be safe. However, that still doesn't mean that we shouldn't be encouraging people to use the new module instead of the old ones. If the new module is only supposed to be used in parallel with the old ones, but feature-wise is identical, then we don't need this module. We think that this new module should be the future for com_content modules, so we should advertise it in our core product by using it as a direct replacement for the old ones in our sampledata.

its easy to update the sample data - shouldnt take more than a few minutes

I have modified the Blog Sample Data to display the new module instead of the old ones. The Testing Sample Data need a complete restructuring and I'm not sure if I'm able to do that.

Hello
about installing a PR
I installed this PR with the 2nd method by downloading https://artifacts.joomla.org/drone/j...te_Package.zip then doing the manual update which asks me to identify myself on the site.
I had already tried by copying the update url and putting development version without success
So I easily went from the test version joomla 5.2alpha3 to this version joomla 5.2alpha3+PR new module
It's easier than going through the component whose options I got lost in by using it once a year.

For the future, I have 2 questions,
a) when I install another more recent version, if this pr is not included, what I did will be overwritten?
b) if I put another joomla 5.2x + PR, it will not overwrite and a priori should not interfere?

About the module
Good to know you must set display options correctly" otherwise there is no display
I appreciate

• the presentation on the same line. I imagine that after the templates including astroid will be able to create a nice layout :-)
• the fact of clicking on a block

Afterwards I have 2 questions
c) it seems that the images are not rendered! yet I put 2 nice images in the content of the text.
Well it is less important for me because I systematically use the Tassos custom field images which will probably never be taken natively in this module
d) I have not seen any repercussions of modifying the parameter: "Articles Info Layout​"

Sorry if my bad English is poorly translated.

@HDInfautre sorry I don't understand your problems / questions about the installation ...

Article Info Layout = Multiline

Article Info Layout = Single Line

Article image: first image is intro image, secon one is an image inside the article (before the read more)

I have no idea what Astroid framework will do with this module ...

HI
hanks for this very quick response :-)
I'm giving up my positive remarks or on astroid.
I'm just focusing on the remarks
c) it seems that the images are not rendered!
https://j5test-seul.88h.ovh/index.php/test-mod yet I put 2 beautiful images in the text content.
Well it's less important for me because I systematically use the Tassos custom field images which will probably never be taken natively in this module
d) I have not seen any repercussions of modifying the parameter: "Articles Info Layout​"

After possibly on the questions concerning the post-patch updates if it's understandable!?

Make sure this setting is set to show

The images are not rendered because you have set a character limit for the introtext ... For my understanding that is correct, but maybe I need to change the visibility of the "Show Article Images" parameter...

You are showing only one element (category) from the article info that is why you don't see any difference between multiline and single line, try displaying the author too

HI
hanks for this very quick response :-)
I'm giving up my positive remarks or on astroid.
I'm just focusing on the remarks
c) it seems that the images are not rendered!
https://j5test-seul.88h.ovh/index.php/test-mod yet I put 2 beautiful images in the text content.
Well it's less important for me because I systematically use the Tassos custom field images which will probably never be taken natively in this module
d) I have not seen any repercussions of modifying the parameter: "Articles Info Layout​"

After possibly on the questions concerning the post-patch updates if it's understandable!?

The images are not rendered because you have set a character limit for the introtext ... For my understanding that is correct, but maybe I need to change the visibility of the "Show Article Images" parameter...

You are showing only one element (category) from the article info that is why you don't see any difference between multiline and single line, try displaying the author too

Hi
thanks for the follow-up
The image is at the beginning of the text
I checked what seems right to me.

I can give access if necessary.

Again: if you set a limit for the introtext the article images will not be rendered. I commited a new version of the module, I changed the parameter, "Show Article Images" will only be displayed if the introtext limit is set to 0 (not truncated)

Again: if you set a limit for the introtext the article images will not be rendered. I commited a new version of the module, I changed the parameter, "Show Article Images" will only be displayed if the introtext limit is set to 0 (not truncated)

OK, sorry, I misunderstood.
It works as expected, but as you can see from the result https://j5test-seul.88h.ovh/index.php/test-mod , I think people will prefer to have an image AND reduced text :-)
Otherwise I imagine you'd have to overload the image, which I'm not sure is possible for many people.
I'll be able to do it myself, because you're not going to support images in a custom field anyway?
Regards

You should use intro or full image from the article
And no, we can't support custom fields in the module, for that you will need an override

Hi
do you plan to put "deprecated" in the name of the old modules to avoid that people continue to use the old modules!?

Hi do you plan to put "deprecated" in the name of the old modules to avoid that people continue to use the old modules!?

why would you want to do that

why would you want to do that

Hey I expressed myself badly!?
The new module replaces the old ones in a much better way. The idea is that people after the release of Joomla 5.2 are encouraged to use the new module :-) and this sends a signal to people who pass by. I see this in Joomla extensions (Tassos) as well as other computer software

But of course, the old sites would use them for years and years

So soooo happy that #18455 is finally to become reality :-) Thank you so much @drmenzelit

Oh, I think I did a mistake in some commit ... the display options should be set to "Title Only (lists) = Yes". I corrected it now.

Hey there,

I'm working on documentation for this module. While testing I am noticing that when the Display Option Show Intro/Full Image is set to show an image, and the article has an image but no caption for that image, an error message appears at the top of the module.

For example, this Cradle Mountain article has no caption defined for either its intro or full image:

@JeffOnWire I can't reproduce the warnings. Can you please check the content of lines 281 and 288?

I have tested this item ✅ successfully on 56845d4

Hi
Already successfully tested with alpha3 following my comments on github.
I've just re-tested the package with the latest version.

With this module you can replace the existing mod_articles_xx.

This mean the 5 Article Modules you listed below? I tried to show related Articles but didn't get it to work.

@fgsw yes, only the listed ones. Articles Related is still an own module.

I have tested this item ✅ successfully on 56845d4

I have successfully tested this.

I will add these comments:

a. if I set the Category Filtering Type to Exclusive and I do NOT select a Category, I get results?
(i.e. I would have expected an error saying that I had not selected a category) - Viviana explained that this is taken from mod_articles_category and there is no error telling you you need to select a category but to me it feels inconsistent with the rest of Joomla so many in the future that could happen.

b. If you choose under Filtering Options: Archived Articles --> Only and you don't happen to have any the Module completely disappears vs showing the Module with a statement like: No articles to display. As per Viviana, the same behavior as the current Archived Articles module but again, it doesn't feel homogeneous will Joomla as a whole - hopefully in the future it will be updated/more homogeneous.

This is a GREAT module - great work by Viviana and Brian (and whomever else helped if anyone did), but as it has so many options, capabilities, it is a bit daunting so great documentation (or possibly additionally a Guided Tour as well) will be needed to make it easiest for users and to guarantee (wide) adoption.

Articles Related is still an own module.

@drmenzelit Thanks for the answer. Your PR makes Joomla easier to use.

I've restored the previous human test results in the issue tracker as the change which invalidated the test count was just a rename of the update SQL script for PostgreSQL, which does not have any functional impact as that is the only 5.2 update SQL yet so it would have run despite of the wrong year in the file name.

=> RTC

@drmenzelit After further testing this is maybe not an issue that requires attention. I am able to fix the problem by opening an article for edit, then saving it without making any changes. Possibly the issue is related to sample data? (Perhaps the sample data has a null value for the captions, or the captions property is not initially defined in the database. Opening the article in the editor and saving it maybe updates the property in the database in a way that fixes the problem—if that makes any sense).

@JeffOnWire thats a bug - you should not need to do that - although I do understand why that works

@exlemor Notes that the Articles module disappears completely when there is no content, and this happens also with the Articles - Archive module. I noticed this as well and agree these modules should always appear with a "no articles to display" message.

Yes that's a bug in the sample data plugin. Please @JeffOnWire could you open the issue tracker and mark the test as "not successful"? Then it is no longer Ready To Commit (RTC) and @drmenzelit can work on it. Thank you for testing so carefully :)

Back to pending due to previous comment.

Yes that's a bug in the sample data plugin. Please @JeffOnWire could you open the issue tracker and mark the test as "not successful"? Then it is no longer Ready To Commit (RTC) and @drmenzelit can work on it. Thank you for testing so carefully :)

I've already removed RTC.

Here's what I've done just now to recreate the issue.

Saved configuration.php then removed it from the root.
Created a new database.
Opened the site in a browser and configured the site using the new database.
Logged in as admin.
From the administrator dashboard I see a Sample Data module.
Click on the first option to install Testing Sample Data
Install the Articles module and configure:

• Title: Articles Module Test
• Position: sidebar-right
• Article IDs to Include: 11 and 25
• Display Options > Show Intro/Full Image: Intro Image

And then viewing the homepage I see the errors:

I have modified the Blog Sample Data to display the new module instead of the old ones. The Testing Sample Data need a complete restructuring and I'm not sure if I'm able to do that.

As I wrote before, I did not touched the Testing Sample Data Plugin and I'm not able to rewrite it.

I will try to reproduce the warnings, but if the problem comes from the Testing Sample Data Plugin, then someone else has to repair it.

The bug with the caption is related with the Testing Sample Data and not with this PR. The same problem appears with an Articles Newsflash module on a site with installed sample data

Testing sample data are not in scope for this PR, but we don't know if older content has empty values in images. Code Review looks good for me.

I modified the query to catch the possibility of a wrong set image caption. @HDInfautre , @exlemor , @JeffOnWire I will appreciate a new test from you :-)

@drmenzelit The error is gone. Everything looks fine to me.

@JeffOnWire can you please mark your test in the issue tracker? Thanks

I have tested this item ✅ successfully on 56845d4

The error messages I received before no longer appear. Module is working correctly.

When I set the Filter Options to show featured articles, and set the Display Options to show Introtext (with Introtext Limit (Characters) set to 100 characters), the featured articles are displayed with all the article text (or up to the Read More placeholder if the article has one). Should the amount of text for featured articles be restricted to Introtext Limit (Characters) like any other article?

When I set the Filter Options to show featured articles, and set the Display Options to show Introtext (with Introtext Limit (Characters) set to 100 characters), the featured articles are displayed with all the article text (or up to the Read More placeholder if the article has one). Should the amount of text for featured articles be restricted to Introtext Limit (Characters) like any other article?

I could not reproduce this issue. Please check again and make sure you tell us how to reproduce this, so we can check/fix any issues.

RTC

@RickR2H this should not be RTC until @JeffOnWire responds to your question

@brianteeman I later noticed that @JeffOnWire tested successfully.

Merged the changes done by @bembelimen (thank you for cleaning the code) and corrected an error in the logic of displaying article imagas that was interfering with the truncation of the intro text.

@drmenzelit Could you fix PHPCS errors reported here https://ci.joomla.org/joomla/joomla-cms/78353/1/7 ? Thanks in advance.

@richard67 I was on it, done ... I hope ;-)

Hello,
tested in Multilanguage environment (FR/ENG), very interesting feature. Works well. One notice : "Article IDs to Exclude" - I did not manage how to separate the differents articles, tested, comma (,), dash (-) ... until I saw the multiline icon to show it needs a carriage return. It would be nice just to add an info at the RollOver on the "Article IDs to Exclude" text which would say "article ID separated by a carriage return" .
Best
Philippe

I have tested this item ✅ successfully on 56845d4

Hello,
tested in Multilanguage environment (FR/ENG), very interesting feature. Works well. One notice : "Article IDs to Exclude" - I did not manage how to separate the differents articles, tested, comma (,), dash (-) ... until I saw the multiline icon to show it needs a carriage return. It would be nice just to add an info at the RollOver on the "Article IDs to Exclude" text which would say "article ID separated by a carriage return" .
Best
Philippe

@RickR2H I tested this morning and was unable to recreate the introtext issue I raised on August 23. I have no issues. Everything looks fine.

I'm working with Ordering Options today and may have another issue—order by alias seems to be ordering by title. Here's my setup:

Removed configuration.php from the root.
Created a new database, configured Joomla and logged in as admin.
From the administrator dashboard I see a Sample Data module. Installed Testing Sample Data
Created two new user accounts, for Arthur Able and Peter Petra.
Went to Content > Articles, sorted by ID ascending and edited articles with ID 1-6

• Administrator Components (ID 1): Created by = Arthur Able; leave alias blank
• Archive Module (ID 2): Alias = Barbara Brown
• Article Categories Module (ID 3): Created by = Peter Petra; leave alias blank
• Articles Category Module (ID 4): Alias = Eddie Earnest
• Authentication (ID 5): No change (created by you); leave alias blank
• Australian Parks (ID 6): Alias = Helen Hurry

Install the Articles module and configure:

Title: Articles Module Test
Articles to Display: 6
Position: sidebar-right
Article IDs to Include: 1,2,3,4,5,6 on separate lines

Display Options: Title Only = No; Author: Show

Ordering Options: Alias Ascending (and switched back and forth between Ascending and Descending while testing)

I'm observing that the articles are always ordered by Title, regardless of the Author or Alias value.

Question about how Author and Alias

In Display Options we have Author (which will display the Created by Alias if defined in the article, otherwise it displays the Created by user), but in Ordering Options we have Alias, which I'm assuming would function in a similar way...ordering by either Created by or Created by Alias. Should these be labeled the same, either Author or Alias, to be consistent?

Finish Publishing Date

In Ordering Options we have the option to order by Finish Publishing Date but there is no display option for the finish publishing date. Should there be?

@JeffOnWire I think there is a misunderstanding of Alias ... Alias refers to the alias of the article, this one on the right:

@drmenzelit Ah, thank you.

Thank you @drmenzelit for this great feature!

Sorry, so it doesn't get lost, I created #44088.

@Kostelano thank you for the hint, we removed some parameters on the last version of the module before the merge and it seems we forgot to chech this one. We will correct it.