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

Splitting the large GN stylesheet into smaller ones and speed up loading #4044

Merged
merged 1 commit into from
Sep 23, 2019

Conversation

MichelGabriel
Copy link
Contributor

This PR splits the large GeoNetwork stylesheet in smaller stylesheets.

This is done in order to speed up the loading of the different files. A browser can load these smaller files simultaneously. The first styles needed to draw the opening page are accessible faster. The page doesn't have to wait until the one large stylesheet is loaded.

The new GN stylesheet is now (more or less) half the size of the old one.

@MichelGabriel MichelGabriel changed the title Splitting the large GN stylesheet into smaller ones Splitting the large GN stylesheet into smaller ones and speed up loading Sep 19, 2019
@fxprunayre fxprunayre added this to the 3.8.1 milestone Sep 23, 2019
@fxprunayre
Copy link
Member

The new GN stylesheet is now (more or less) half the size of the old one.

Before
image

After
image

ok but the total is the same. The focus is mainly to reduce the size of gn_search_default to have faster rendering of the base styles, right ?

@MichelGabriel
Copy link
Contributor Author

Yes, that's correct

@fxprunayre fxprunayre merged commit 5d77d09 into geonetwork:master Sep 23, 2019
@import "ellipsis.less";
@import "timeline.less";
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is removed but not added anywhere else so layout is broken in record history, harvester history

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

image

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Should we add it back in gn-picker.less ?

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Fixed in #4128

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants