?
avatar yasirunilan
yasirunilan
19 Mar 2017

Steps to reproduce the issue

In the main menu select Media and click it.

Expected result

Items in the page should be in a order with spacing

Actual result

layout

System information (as much as possible)

Joomla! 4.0.0-dev
Windows 10
Chrome 57.0.2987.110 (64-bit)
PHP 5.6

Additional comments

avatar yasirunilan yasirunilan - open - 19 Mar 2017
avatar joomla-cms-bot joomla-cms-bot - change - 19 Mar 2017
Labels Added: ?
avatar joomla-cms-bot joomla-cms-bot - labeled - 19 Mar 2017
avatar franz-wohlkoenig
franz-wohlkoenig - comment - 19 Mar 2017

known Issue #14039

avatar Bakual
Bakual - comment - 19 Mar 2017

@wilsonge Maybe it would be great to have a "Known issue" list somewhere public. So people know what to expect when trying out J4.
Eg "Template still work in progress" and "MediaManager gets a complete redesign".

avatar franz-wohlkoenig
franz-wohlkoenig - comment - 19 Mar 2017

@Bakual would show a Message in 4.0 like Patchtester shows.

avatar mbabker
mbabker - comment - 19 Mar 2017

Patch tester is easy to put a message in. And I honestly put it there not because the code is broken now, but because since it is a moving target still, there's no guarantee it won't be broken.

Putting those messages in core would add a lot of maintenance overhead. Any known issues should easily be found with either GitHub's or our own issue tracker's search/filters.

avatar franz-wohlkoenig
franz-wohlkoenig - comment - 19 Mar 2017

@mbabker less People search before report an issue. Is an Info at "Media" an easy Solution?
bildschirmfoto 2017-03-19 um 09 08 44

avatar yasirunilan
yasirunilan - comment - 19 Mar 2017

@franz-wohlkoenig I think that's better.

avatar mbabker
mbabker - comment - 19 Mar 2017

It is easy to put whatever notifications we want into core, but that doesn't mean it's appropriate. And to be honest it is not appropriate to ever check into the code a warning mechanism that essentially says "this section is a work in progress, see X issue for more details".

avatar franz-wohlkoenig
franz-wohlkoenig - comment - 19 Mar 2017

@mbabker i understand, on other Hand we get known Issues reported. If an "Known issues"-List is the easy Solution, i'm fine with it.

avatar C-Lodder
C-Lodder - comment - 19 Mar 2017

The new media manager has not yet been merged, so I'm not going to bother styling something that'll be replaced.

This can be closed.

avatar wilsonge
wilsonge - comment - 19 Mar 2017

I'm going to leave this open. But don't expect much progress, there's no need to do this for now whilst @laoneo 's team is working on the revamped media manager. If it looks like that won't make it for J4 for whatever reason it's not a huge amount of work to go back style that

avatar joomla-cms-bot joomla-cms-bot - edited - 19 Mar 2017
avatar franz-wohlkoenig franz-wohlkoenig - change - 30 Mar 2017
Category com_media Layout
avatar joomla-cms-bot joomla-cms-bot - change - 30 Mar 2017
Title
[4.0] Layout issue in Media
[4.0] Layout issue in Media
avatar franz-wohlkoenig
franz-wohlkoenig - comment - 3 Apr 2017

@wilsonge shouldn't this be closed cause of https://developer.joomla.org/roadmap.html?


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

avatar franz-wohlkoenig franz-wohlkoenig - change - 3 Apr 2017
Status New Discussion
avatar mbabker
mbabker - comment - 3 Apr 2017

No. Legitimate issues should not be closed just because something is a known work in progress or planned to be replaced. Because if the plan to replace it doesn't get followed up on for whatever reason, then this has to be acted upon. It's a low priority issue, but one nonetheless.

avatar franz-wohlkoenig
franz-wohlkoenig - comment - 3 Apr 2017

thanks for Info, @mbabker

avatar C-Lodder
C-Lodder - comment - 3 Oct 2017

@franz-wohlkoenig - The new Media Manager is now gradually being merged into J4, so this can be closed

avatar brianteeman brianteeman - change - 3 Oct 2017
Status Discussion Closed
Closed_Date 0000-00-00 00:00:00 2017-10-03 13:02:32
Closed_By brianteeman
avatar brianteeman brianteeman - close - 3 Oct 2017
avatar brianteeman
brianteeman - comment - 3 Oct 2017

Closed as requested

Add a Comment

Login with GitHub to post a comment