Bug 20427

Summary: Convert OPAC LESS to SCSS
Product: Koha Reporter: Owen Leonard <oleonard>
Component: OPACAssignee: Owen Leonard <oleonard>
Status: CLOSED FIXED QA Contact: Testopia <testopia>
Severity: enhancement    
Priority: P5 - low CC: jonathan.druart, katrin.fischer, martin.renvoize, mtj, nick
Version: Main   
Hardware: All   
OS: All   
URL: https://wiki.koha-community.org/wiki/Working_with_staff_client_SCSS
See Also: https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=1918
Change sponsored?: --- Patch complexity: Medium patch
Documentation contact: Documentation submission:
Text to go in the release notes:
Version(s) released in:
Bug Depends on: 19474    
Bug Blocks: 20554, 31528    
Attachments: Bug 20427: Convert OPAC LESS to SCSS
Bug 20427 (follow-up) Compiled CSS
Bug 20427: Convert OPAC LESS to SCSS
Bug 20427 (follow-up) Compiled CSS
Bug 20427: Convert OPAC LESS to SCSS
Bug 20427 (follow-up) Compiled CSS
Bug 20427: Convert OPAC LESS to SCSS
Bug 20427 (follow-up) Compiled CSS
Bug 20427: Convert OPAC LESS to SCSS
Bug 20427: Convert OPAC LESS to SCSS
Bug 20427: Convert OPAC LESS to SCSS
Bug 20427: Convert OPAC LESS to SCSS
Bug 20427: (follow-up) Bring SCSS up to date with latest LESS changes

Description Owen Leonard 2018-03-15 15:44:52 UTC
Since we're working to add SCSS to the staff client (Bug 19474), we can convert the OPAC to use SCSS too and use the same build process for both.
Comment 1 Owen Leonard 2018-03-16 09:45:55 UTC Comment hidden (obsolete)
Comment 2 Owen Leonard 2018-03-16 09:45:59 UTC Comment hidden (obsolete)
Comment 3 Owen Leonard 2018-03-16 12:37:04 UTC Comment hidden (obsolete)
Comment 4 Owen Leonard 2018-03-16 12:37:08 UTC Comment hidden (obsolete)
Comment 5 Claire Gravely 2018-03-20 09:28:10 UTC Comment hidden (obsolete)
Comment 6 Claire Gravely 2018-03-20 09:28:14 UTC Comment hidden (obsolete)
Comment 7 Katrin Fischer 2018-03-30 11:26:51 UTC
Hi Owen, can you please rebase 19474 and this on top so I can test? Thx!
Comment 8 Owen Leonard 2018-03-30 12:14:10 UTC Comment hidden (obsolete)
Comment 9 Owen Leonard 2018-03-30 12:16:45 UTC Comment hidden (obsolete)
Comment 10 Katrin Fischer 2018-03-30 13:01:27 UTC Comment hidden (obsolete)
Comment 11 Katrin Fischer 2018-03-30 13:02:33 UTC Comment hidden (obsolete)
Comment 12 Owen Leonard 2018-07-23 17:33:36 UTC Comment hidden (obsolete)
Comment 13 Owen Leonard 2018-07-23 18:53:38 UTC
Created attachment 77193 [details] [review]
Bug 20427: Convert OPAC LESS to SCSS

This patch converts the OPAC LESS files to SCSS. In the process of
converting opac.less (and other .less files) to .scss, I have improved
the completeness of the nesting, reordered properties, and in general
done cleanup based on rules in .scss-lint.yml. All of these changes
should have no impact on the style of the OPAC.

This patch modifies the commands used to compile CSS so that OPAC and
staff assets can be processed separately:

'yarn build' <-- Builds the staff client assets by default
'yarn build --view opac' <-- Builds OPAC assets

To test, apply the patch and compile the CSS by running the command
above (or apply the follow-up patch with compiled CSS).

Do a thorough review of the OPAC, taking care to cover as many pages as
possible and checking responsive behavior at the same time. There should
be no visual differences in the OPAC before and after applying the
patch.

Signed-off-by: Claire Gravely <claire.gravely@bsz-bw.de>

Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>
Comment 14 Owen Leonard 2018-08-09 14:16:19 UTC
Created attachment 77628 [details] [review]
Bug 20427: (follow-up) Bring SCSS up to date with latest LESS changes

This patch catches the SCSS up to the latest changes in master:

Changes to opac.less:

Bug 20756
Bug 20559
Bug 7547

Changes to print.less:

Bug 16575
Comment 15 Nick Clemens 2018-08-09 15:26:24 UTC
Awesome work all!

Pushed to master for 18.11
Comment 16 Martin Renvoize 2018-08-10 08:22:56 UTC
Enhancement, this will not be backported for the 18.05.x series.