? ? Pending

User tests: Successful: Unsuccessful:

avatar shamsbd71
shamsbd71
1 Dec 2015

Joomla has the ability to set the articles details page layout from category menu item. but it wasn't implemented. just add these xml and its gonna work perfect.

Before

image

After

image

avatar shamsbd71 shamsbd71 - open - 1 Dec 2015
avatar shamsbd71 shamsbd71 - change - 1 Dec 2015
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 1 Dec 2015
Labels Added: ?
avatar ghazal
ghazal - comment - 3 Dec 2015

Test OK.
I hadn't figured out it was missing. But actually, it is.
TKS.

avatar brianteeman brianteeman - change - 28 Mar 2016
Category Components
avatar mikeveeckmans mikeveeckmans - test_item - 30 Mar 2016 - Tested successfully
avatar mikeveeckmans
mikeveeckmans - comment - 30 Mar 2016

I have tested this item :white_check_mark: successfully on 96822b7

TEST OK


This comment was created with the J!Tracker Application at issues.joomla.org/joomla-cms/8579.

avatar designbengel designbengel - test_item - 8 Apr 2016 - Tested unsuccessfully
avatar designbengel
designbengel - comment - 8 Apr 2016

I have tested this item :red_circle: unsuccessfully on 96822b7

I added a cuatom template override and it doesn´t appear in the list.


This comment was created with the J!Tracker Application at issues.joomla.org/joomla-cms/8579.

avatar designbengel
designbengel - comment - 9 Apr 2016

Btw: We tested also when adding an override for an article, and that worked. But that seems not logical to me, because i expect to be able to choose a category list override there?
Maybe we need then two fields there 1) List view layout and 2) Article view layout


This comment was created with the J!Tracker Application at issues.joomla.org/joomla-cms/8579.

avatar webnet-assmann webnet-assmann - test_item - 9 Apr 2016 - Tested successfully
avatar webnet-assmann
webnet-assmann - comment - 9 Apr 2016

I have tested this item :white_check_mark: successfully on 96822b7

Alternative Layout is working for Articles. It's a bit confusing. It should be called Alternative Layout for Articles.


This comment was created with the J!Tracker Application at issues.joomla.org/joomla-cms/8579.

avatar webnet-assmann
webnet-assmann - comment - 9 Apr 2016

Alternative Layout is working for Article. It's a bit confusing. It should be called Alternative Layout for Articles.


This comment was created with the J!Tracker Application at issues.joomla.org/joomla-cms/8579.

avatar joomla-cms-bot
joomla-cms-bot - comment - 10 Apr 2016

This PR has received new commits.

CC: @designbengel, @mikeveeckmans, @webnet-assmann


This comment was created with the J!Tracker Application at issues.joomla.org/joomla-cms/8579.

avatar shamsbd71
shamsbd71 - comment - 10 Apr 2016

Yes i agree that if we change the label to Alternative Layout for Articles would be more clear.
Need to add the language then. I'll add the correct language.

avatar joomla-cms-bot
joomla-cms-bot - comment - 10 Apr 2016

This PR has received new commits.

CC: @designbengel, @mikeveeckmans, @webnet-assmann


This comment was created with the J!Tracker Application at issues.joomla.org/joomla-cms/8579.

avatar joomla-cms-bot
joomla-cms-bot - comment - 10 Apr 2016

This PR has received new commits.

CC: @designbengel, @mikeveeckmans, @webnet-assmann


This comment was created with the J!Tracker Application at issues.joomla.org/joomla-cms/8579.

avatar joomla-cms-bot joomla-cms-bot - change - 10 Apr 2016
Labels Added: ?
avatar brianteeman brianteeman - change - 27 Apr 2016
Category Components Components Language & Strings
avatar brianteeman brianteeman - change - 27 Apr 2016
Labels
avatar conconnl
conconnl - comment - 28 Jun 2016

@shamsbd71 are all the comments integrated in the patch?
If so, I and hopefully others can test this PR.


This comment was created with the J!Tracker Application at issues.joomla.org/joomla-cms/8579.

avatar shamsbd71
shamsbd71 - comment - 28 Jul 2016

@conconnl yes it is.
thanks

avatar schmidtpaddy schmidtpaddy - test_item - 1 Aug 2016 - Tested unsuccessfully
avatar schmidtpaddy
schmidtpaddy - comment - 1 Aug 2016

I have tested this item ? unsuccessfully on cdf7dc4

field alternative layout didn't appear @icampus


This comment was created with the J!Tracker Application at issues.joomla.org/joomla-cms/8579.

avatar shamsbd71
shamsbd71 - comment - 1 Aug 2016

@schmidtpaddy there are two files, please try both. you have applied only language file changes.
https://github.com/joomla/joomla-cms/pull/8579/files
thanks

avatar Schmidie64
Schmidie64 - comment - 2 Aug 2016

tested:

  • install plugin successfully
  • everything worked well @icampus
avatar Schmidie64 Schmidie64 - test_item - 2 Aug 2016 - Tested successfully
avatar Schmidie64
Schmidie64 - comment - 2 Aug 2016

I have tested this item successfully on cdf7dc4

tested:
•install plugin successfully
•everything worked well @icampus


This comment was created with the J!Tracker Application at issues.joomla.org/joomla-cms/8579.

avatar roland-d
roland-d - comment - 2 Aug 2016

@schmidtpaddy Can you test this once more please? Thanks.


This comment was created with the J!Tracker Application at issues.joomla.org/joomla-cms/8579.

avatar Sieger66
Sieger66 - comment - 15 Aug 2016

Alternative Layout for Article is Display and is a good feature but
I have testet #8579 the "Alternative Layout for Article" and see this problem (with and without SEO):

For example go to article-category-list-menuitem A -> Details -> Template Style and change to an other Template with Template-Override
( with a not default name of the override(unique name) ; for example: /html/com_content/article/my-66-override.php )

then go to List Layouts -> Alternative Layout for Article and see the not correct list in.
Then click the "Save"-Button and go Back to List Layouts -> Alternative Layout for Article and see the correct list now.

Now select this override "my-66-override" and save it.
Now the PR works as expected but if you now go to
article-category-list-menuitem A -> Details -> Template Style and change to an other Template and save it.

Then works the override furthermore.

If you now go to the article-category-list-menuitem to List Layouts -> Alternative Layout for Article and change this to "From Component -> default" and save it then is come the expected result(view without the override from the other Template).

Similar problem as in my post in #8578

I hope this is helpful.


This comment was created with the J!Tracker Application at issues.joomla.org/joomla-cms/8579.

avatar shamsbd71
shamsbd71 - comment - 16 Aug 2016

@Sieger66

You make two article-category-list-menuitems A B
both for the same article-category

This is wrong! you should not create the same menu twice and it's not practical either.

If you check Joomla routing, you are creating confusion for the routing not for the menu item.

This setting works based on menu item id. so when you have duplicate menu itemid for the same category, you are confusing yourself :(

thanks

avatar Sieger66
Sieger66 - comment - 16 Aug 2016

But the Problem in my first post in this PR is a problem that comes with this PR.

To solve this it is additional or changed code required.

avatar microtribe
microtribe - comment - 13 Nov 2016

I have tested this item successfully on cdf7dc4

I tested this - first to confirm that the Alternative Layout selector was not available. Then I used the Patch Tester to install the patch. Then created a Category List menu item. Then check to see if the Alternative Layout selector was visible - it was. Then I created an article alternative layout override. Then I visited the Category List menu item, and the Alternative Layout I created was also visible and selectable.


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/8579.

avatar microtribe microtribe - test_item - 13 Nov 2016 - Tested successfully
avatar franz-wohlkoenig
franz-wohlkoenig - comment - 6 Jan 2017

I have tested this item ? unsuccessfully on cdf7dc4

All works like @microtribe wrote above, but "Show Votes in List" and "Show Ratings in List" isn't shown anymore after Patch.

Before Patch

before

After Patch

after


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/8579.
avatar franz-wohlkoenig franz-wohlkoenig - test_item - 6 Jan 2017 - Tested unsuccessfully
avatar franz-wohlkoenig franz-wohlkoenig - change - 6 Apr 2017
The description was changed
Status Pending Needs Review
avatar joomla-cms-bot joomla-cms-bot - change - 6 Apr 2017
Category Components Language & Strings Administration Language & Strings Front End com_content Components
avatar joomla-cms-bot joomla-cms-bot - edited - 6 Apr 2017
avatar mbabker
mbabker - comment - 21 May 2017

This PR needs to be synchronized with the current development branches and the issues addressed for it to be reviewed.

avatar franz-wohlkoenig franz-wohlkoenig - change - 22 May 2017
Status Needs Review Information Required
avatar franz-wohlkoenig
franz-wohlkoenig - comment - 27 May 2017

If this PR get no Response, it will be closed at 22th June 2017.


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/8579.

avatar coolcat-creations
coolcat-creations - comment - 27 May 2017

How to sync it? should i create a new PR on the latest staging and reference it?

avatar brianteeman
brianteeman - comment - 27 May 2017

@coolcat-creations yes that will be the easiest in this case

avatar brianteeman
brianteeman - comment - 18 Jun 2017

closed please see #16755

avatar brianteeman brianteeman - change - 18 Jun 2017
Status Information Required Closed
Closed_Date 0000-00-00 00:00:00 2017-06-18 19:54:28
Closed_By brianteeman
avatar brianteeman brianteeman - close - 18 Jun 2017

Add a Comment

Login with GitHub to post a comment