Print style updates #810

Merged
merged 3 commits into from Oct 9, 2014

Conversation

Projects
None yet
2 participants
@skoolbus39
Member

skoolbus39 commented Oct 6, 2014

Fixes #585

@kabel

This comment has been minimized.

Show comment
Hide comment
@kabel

kabel Oct 8, 2014

Contributor

Excellent work! One suggestion I might put in is a reset for the maincontent width:

#maincontent > * {
    .reset();
    max-width: none;
}
Contributor

kabel commented Oct 8, 2014

Excellent work! One suggestion I might put in is a reset for the maincontent width:

#maincontent > * {
    .reset();
    max-width: none;
}
+ box-shadow: none;
+ border-bottom: 1px solid black;
+
+ .wdn-inner-wrapper:first-of-type:after {

This comment has been minimized.

@kabel

kabel Oct 8, 2014

Contributor

I think this is used to insert a physical line-break after the institution title, but this hack doesn't work on IE. I would recommend removing this selector and removing the float from the institution title to force the site title to its own line by way of block layout.

@kabel

kabel Oct 8, 2014

Contributor

I think this is used to insert a physical line-break after the institution title, but this hack doesn't work on IE. I would recommend removing this selector and removing the float from the institution title to force the site title to its own line by way of block layout.

kabel added a commit that referenced this pull request Oct 9, 2014

@kabel kabel merged commit 24713e2 into unl:develop Oct 9, 2014

1 check passed

continuous-integration/travis-ci The Travis CI build passed
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment