Skip to content
This repository has been archived by the owner on Mar 30, 2024. It is now read-only.

Hide cookie notice panel in printed output. #17

Open
wants to merge 3 commits into
base: main
Choose a base branch
from

Conversation

joedolson
Copy link

Description

Hides the cookie notice when printing the web page.

How Has This Been Tested?

CSS only change; tested in browser only. Test run: https://github.com/joedolson/wp-gdpr-cookie-notice/actions

Screenshots (jpeg or gifs if applicable):

Not applicable. Change is the absence of the cookie notice.

Types of changes

I'd consider this a bug fix, as it removes an obstruction of content in print views.

Checklist:

  • [x ] My code is tested.
  • [ x] My code is backward-compatible with WordPress 4.9.6 and PHP 7.0.
  • [ x] My code follows the WordPress coding standards.
  • [x ] My code has proper inline documentation.

@joedolson
Copy link
Author

Whoops. I only reverted and re-ran that to run tests; I'd only intended to include the last commit in the PR. Well, nothing important in those changes, anyway.

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

Successfully merging this pull request may close these issues.

1 participant