?
avatar Harmageddon
Harmageddon
17 Jan 2015

Steps to reproduce the issue

1) Switch your site's default template to beez3 or a third party template.
2) Log in to frontend with a user account, that can edit or create an article in frontend.
3) Create / edit an article in frontend.
4) Click the "insert image" button.

Actual result

screen shot 2015-01-17 at 06 10 21

Expected result

screen shot 2015-01-17 at 06 10 23

Reported by user MrMusic in a german Joomla! Forum (thread). I can confirm this. I'm not sure, but I think this worked before. On first glance, the file media/jui/less/thumbnails.less or media/jui/css/thumbnails.css is missing here.

Tested on current master and 3.3.6 with the same result.

Votes

# of Users Experiencing Issue
1/1
Average Importance Score
4.00

avatar Harmageddon Harmageddon - open - 17 Jan 2015
avatar Harmageddon Harmageddon - change - 17 Jan 2015
Status New Confirmed
avatar brianteeman brianteeman - change - 17 Jan 2015
Labels Added: ?
avatar wilsonge
wilsonge - comment - 29 Jan 2015

I presume you expected and actual images are the wrong way around here?


This comment was created with the J!Tracker Application at issues.joomla.org/joomla-cms/5790.
avatar Harmageddon
Harmageddon - comment - 29 Jan 2015

@wilsonge Yes, you're right. Thank you! I've corrected the original post.


This comment was created with the J!Tracker Application at issues.joomla.org/joomla-cms/5790.
avatar bembelimen
bembelimen - comment - 14 Mar 2015

I looked into this issue and found the following points:

  • the first problem is, that beez does not load bootstrap by default => solution: add com_media to the template parameters
  • then we see, that we cannot see anything: beez supports bootstrap only in the index.php and not in the component.php => add the code to the component.php
$app            = JFactory::getApplication();
$templateparams = $app->getTemplate(true)->params;
$bootstrap      = explode(',', $templateparams->get('bootstrap'));
$jinput         = JFactory::getApplication()->input;
$option         = $jinput->get('option', '', 'cmd');

if (in_array($option, $bootstrap))
{
    JHtml::_('bootstrap.framework');

    // Load optional rtl Bootstrap css and Bootstrap bugfixes
    JHtml::_('bootstrap.loadCss', true, $this->direction);
}
  • now we have styled boxes without images. Problem: the default bootstrap (from the media folder) does not know icon-folder-2 => we have to rename all icon-folder-2 to icon-folder-close
  • now the li-elements are not in one line => add pull-left to the li elements or add ".thumbnails > li" to the beez css
  • last but not least we have to style the link hover color
avatar bembelimen
bembelimen - comment - 14 Mar 2015

Other solution: put every style property to the beez template css (like icon, etc.) to use it without any bootstrap stuff

avatar chmst
chmst - comment - 14 Mar 2015

As much as I see in the german forums, it would be a good idea to remove the beez template completely fom the core - there are too many questions and problems around it.

avatar AndyGaskell
AndyGaskell - comment - 10 Nov 2015

I think it'd be good to find a solution to this.

It means that using the media manager when front-end-editing is dependent on the template having the required bootstrap bits. This is a use case that many template developers do not test against I think.

Perhaps com_media, when loaded via an iFrame, should have all the styles it needs loaded from within the component.

So, I'm thinking, perhaps this is less of an issue with beez, and more of an issue in that com_media does not include all the styles that it needs to function nicely.

Just some thoughts, I'll have a ponder and see if I can put together some code for this.

avatar dgt41
dgt41 - comment - 10 Nov 2015

Just a hint:
Remove from administrator/components/com_media/views/images/tmpl/default.php

<div class="tabbable">
    <ul class="nav nav-tabs">
        <li class="active"><a href="#browse_tab" data-toggle="tab"><?php echo JText::_('COM_MEDIA_BROWSE_FILES') ?></a></li>
        <li><a href="#upload_tab" data-toggle="tab"><?php echo JText::_('COM_MEDIA_UPLOAD') ?></a></li>
    </ul>

    <div class="tab-content">

as well as lines 147-148 and lines 162-163. This will bring back the old (no tabs view ) and therefore should be easier.
That view will be used only by the mootools part of the field, bootstrap should use the new one, tried it with Hathor...

avatar dgt41
dgt41 - comment - 10 Nov 2015
avatar AndyGaskell
AndyGaskell - comment - 10 Nov 2015

Thanks @dgt41 for the pointers, I'll do some reading of that code and ticket.
I need to have a bit more of a dig into it here, it seems like it might be an issue cause locally related to the template CSS not being loaded into the iFrame.
My thinking, initially, was to add the required css to /media/media/css/popup-imagemanager.css but that may not be ideal, I'll dig into it further.

avatar dgt41
dgt41 - comment - 10 Nov 2015

@AndyGaskell another hint: that modal contains two iframes (the first one is the view images and inside that one there is another iframe from the view medialist (where the icons are). So the solution must cover both views...

avatar brianteeman
brianteeman - comment - 7 May 2016

Thank you for submitting this issue/pull request. The media manager is currently being completely rewritten and I have notified the development team about this issue for their consideration. I am closing this here at this time.


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

avatar brianteeman brianteeman - close - 7 May 2016
avatar brianteeman brianteeman - change - 7 May 2016
Status Confirmed Closed
Closed_Date 0000-00-00 00:00:00 2016-05-07 15:57:20
Closed_By brianteeman

Add a Comment

Login with GitHub to post a comment