? Success
Related to # 5270

User tests: Successful: Unsuccessful:

avatar victorparanhos
victorparanhos
5 Dec 2014

Fixes Fixes image logo in isis as reported on ticket #5270.
This is my first pull-request to a project. If I need to do anything different, tell me.

avatar victorparanhos victorparanhos - open - 5 Dec 2014
avatar jissues-bot jissues-bot - change - 5 Dec 2014
Labels Added: ?
avatar brianteeman
brianteeman - comment - 5 Dec 2014

Thanks for your contribution
As this is a change in the css you need to make it in the less file and
then sue the generatecss script to create the css

On 5 December 2014 at 19:33, Victor Paranhos notifications@github.com
wrote:

Fixes Fixes image logo in isis as reported on ticket #5270
#5270.
This is my first pull-request to a project. If I need to do anything

different, tell me.

You can merge this Pull Request by running

git pull https://github.com/victorparanhos/joomla-cms iss5270

Or view, comment on, or merge it at:

#5330
Commit Summary

  • Fixes image logo in isis - #5270

File Changes

Patch Links:


Reply to this email directly or view it on GitHub
#5330.

Brian Teeman
Co-founder Joomla! and OpenSourceMatters Inc.
http://brian.teeman.net/

avatar brianteeman brianteeman - change - 6 Dec 2014
Category Templates (admin)
avatar brianteeman brianteeman - change - 6 Dec 2014
Rel_Number 5270
Relation Type Related to
avatar brianteeman
brianteeman - comment - 6 Dec 2014

Is this what you intended to do - squash the image to fit the existing space?

screen shot 2014-12-06 at 03 32 08

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

avatar infograf768
infograf768 - comment - 6 Dec 2014

PR wrong as trying to change .view-login in rtl.css which is already changed in core/staging.

new css should rather be something like

.header .container-logo img {
    max-height: 32px;
    width: auto;
}

to keep the right proportions.
Could need something else for IE8 though.

avatar victorparanhos victorparanhos - close - 6 Dec 2014
avatar victorparanhos victorparanhos - change - 6 Dec 2014
Status Pending Closed
Closed_Date 0000-00-00 00:00:00 2014-12-06 15:56:19
avatar micker
micker - comment - 8 Dec 2014

thanks !

avatar infograf768
infograf768 - comment - 8 Dec 2014

@micker

new patch (not fully OK) is #5335

Add a Comment

Login with GitHub to post a comment