Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
When printing out a page, the navbar module is also printed out expanded together with all the action links. I suspect only a very few people needs the navbar, action links and tabs printed out?
Comment | File | Size | Author |
---|---|---|---|
#8 | Issue-2227293-by-hass-Remove-navbar-from-print-view.patch | 1.1 KB | hass |
Comments
Comment #1
lsolesen CreditAttribution: 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 CreditAttribution: 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 CreditAttribution: hass commentedThe attached patch is not for Navbar. CNW. We should remove the navbar in print view for sure.
Comment #5
hass CreditAttribution: hass commentedComment #6
hass CreditAttribution: 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 CreditAttribution: 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 CreditAttribution: 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 CreditAttribution: 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 CreditAttribution: hass commentedThe D8 patch has been commited now.
Comment #12
eshta CreditAttribution: eshta commentedYes - it is :-)
Thanks all - it has been committed.
Comment #13
eshta CreditAttribution: eshta commented