Closed (fixed)
Project:
Navbar
Version:
7.x-1.x-dev
Component:
Code
Priority:
Normal
Category:
Bug report
Assigned:
Unassigned
Reporter:
Created:
27 Mar 2014 at 12:36 UTC
Updated:
14 Mar 2015 at 01:44 UTC
Jump to comment: Most recent, Most recent file
Comments
Comment #1
lsolesen commentedComment #2
saltednutThis seems more like a problem with Navbar itself. If we commit this in Ember we are only fixing it for Ember and not everyone that uses the navbar.
is this also a problem in Drupal 8? If so, this should be a D8 issue for the Toolbar that gets backported to Navbar.
I am moving it to the Navbar issue queue for now.
Comment #3
Grayside commented#1892006: Include a print styling for Seven. has been a discussion of a basic print stylesheet for Seven with an emphasis on removing the toolbar.
Comment #4
hass commentedThe attached patch is not for Navbar. CNW. We should remove the navbar in print view for sure.
Comment #5
hass commentedComment #6
hass commentedThis should do the trick. Need to create a patch for it and for RTL and check out D8 again. The second rule is not 100% verified yet.
Comment #7
hass commentedVertical print view is broken in core, too. At least in seven and should be a module style and not theme style.
Comment #8
hass commentedTested this in LTR and RTL and it works. I'm not sure about the intention of some of the CSS classes. Maybe a few less are sufficient.
Comment #9
eshta commented@hass I've submitted a patch to d8 based on your work here. Once that one is put to bed we can add it in here.
Comment #10
hass commentedThe D8 patch has been commited now.
Comment #12
eshta commentedYes - it is :-)
Thanks all - it has been committed.
Comment #13
eshta commented