NPM Resource Changed ? ? Pending

User tests: Successful: Unsuccessful:

avatar N6REJ
N6REJ
4 Nov 2020

Pull Request for Issue #31304

Summary of Changes

sets correct flow for installer header

Testing Instructions

start fresh install ( or rename configuration.php )
examine header notice logo is missing & title is offset to the right.
apply pr
run "npm ci"
examine header again in both mobile and desktop
Notice everything looks nice.

Actual result BEFORE applying this Pull Request

image

image

Expected result AFTER applying this Pull Request

image
image
image

Documentation Changes Required

none

avatar N6REJ N6REJ - open - 4 Nov 2020
avatar N6REJ N6REJ - change - 4 Nov 2020
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 4 Nov 2020
Category Installation NPM Change
avatar N6REJ N6REJ - change - 4 Nov 2020
Labels Added: NPM Resource Changed ?
avatar ceford
ceford - comment - 5 Nov 2020

#31304 mentioned that the question mark was expected to be white. And if the logo is going to be almost invisible, due to low colour contrast, it may as well be missing.


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

avatar N6REJ
N6REJ - comment - 5 Nov 2020

#31304 mentioned that the question mark was expected to be white. And if the logo is going to be almost invisible, due to low colour contrast, it may as well be missing.

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

colors have been changed to white. I forgot to update the images.

avatar ceford
ceford - comment - 5 Nov 2020

I think it still needs some work! It is coming out different on each of my browser compatibility modes (Firefox and Google Chrome on Mac). Blue on some, white on others. Too big on narrow screens. Question mark is blue.

screen shot 2020-11-05 at 16 44 54


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

avatar N6REJ N6REJ - change - 5 Nov 2020
The description was changed
avatar N6REJ N6REJ - edited - 5 Nov 2020
avatar N6REJ
N6REJ - comment - 5 Nov 2020

looks like you didn't run npm ci

avatar ceford ceford - test_item - 6 Nov 2020 - Tested successfully
avatar ceford
ceford - comment - 6 Nov 2020

I have tested this item successfully on 711fb88

Seems I neglected to run npm in my previous comment. It does now look fine.


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

avatar ceford
ceford - comment - 6 Nov 2020

My screenshot with patch applied and npm run, 380px wide viewport:
screen shot 2020-11-06 at 03 32 21


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

avatar particthistle particthistle - test_item - 7 Nov 2020 - Tested successfully
avatar particthistle
particthistle - comment - 7 Nov 2020

I have tested this item successfully on 711fb88

Tested the header with a new build. Looks good.

Desktop
image

Mobile
image


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/31310.
avatar richard67 richard67 - change - 8 Nov 2020
Status Pending Ready to Commit
avatar richard67
richard67 - comment - 8 Nov 2020

RTC


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

avatar chmst chmst - change - 8 Nov 2020
Status Ready to Commit Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2020-11-08 09:08:55
Closed_By chmst
Labels Added: ?
avatar chmst chmst - close - 8 Nov 2020
avatar chmst chmst - merge - 8 Nov 2020
avatar chmst
chmst - comment - 8 Nov 2020

Thanks!

Add a Comment

Login with GitHub to post a comment