Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

PCHR-3424: Import styles from Bootstrapcivihr #377

Merged
merged 3 commits into from
Jul 26, 2018

Conversation

igorpavlov-zz
Copy link
Contributor

@igorpavlov-zz igorpavlov-zz commented Jul 23, 2018

Overview

In compucorp/civihr-employee-portal#523 we remove the Bootstrapcivihr from SSP. This leads to visual regressions as some styles are missing:

  • Top menu: logo in the top left corner blue box is missing
  • Top menu: user profile image is too big and pushes page content down, no dropdown caret
  • Footer: logo is missing

This PR imports the partials containing these styles from Bootstrapcivihr into SSP Theme.

Before

1

After

2

Technical Details

Simply import the partials:

@import 'SCSSROOT/org.civicrm.bootstrapcivihr/scss/components/outside-namespace/logo';
@import 'SCSSROOT/org.civicrm.bootstrapcivihr/scss/components/outside-namespace/user-menu';

Comments

It was discovered, that the files for "fonts" for the logo icons were not working. The logo fonts worked when Bootstrapcivihr was used because it contained correct font files. This was fixed by simply copying the font files from Bootstrapcivihr and replacing the font files in SSP with them.


✅Manual Tests - passed
✅CSS distributive files - included
✅Backstop tests - passed

@igorpavlov-zz igorpavlov-zz changed the title Pchr 3424 remove bootstrapcivihr from ssp PCHR-3424: Import styles from Bootstrapcivihr Jul 23, 2018
@igorpavlov-zz igorpavlov-zz force-pushed the PCHR-3424-remove-bootstrapcivihr-from-ssp branch from b6e1553 to 6f00470 Compare July 23, 2018 16:50
@igorpavlov-zz igorpavlov-zz force-pushed the PCHR-3424-remove-bootstrapcivihr-from-ssp branch from ba8a74e to 8d519bc Compare July 25, 2018 13:39
@igorpavlov-zz igorpavlov-zz force-pushed the PCHR-3424-remove-bootstrapcivihr-from-ssp branch from 8d519bc to 31ba70b Compare July 25, 2018 13:39
@igorpavlov-zz igorpavlov-zz merged commit 5ebdafd into staging Jul 26, 2018
@igorpavlov-zz igorpavlov-zz deleted the PCHR-3424-remove-bootstrapcivihr-from-ssp branch July 26, 2018 12:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants