Comments (9)
I think this issue can be closed. I did not realize that the privacy policy link was already there.
The only thing is that the check on the existence of the privacy policy function might be superfluous if the theme would check the WP version (for only 5+).
There are other issues where adding a nav menu and a widget area are being discussed.
from twentynineteen.
Related to #45
from twentynineteen.
In my opinion it should be an own menu, where the user can decide which points should be displayed. Some countries have stricter rules. In Germany for example we have to have two sites "Impressum" and the "privacy policy".
As far as I understand the suggested code, it would only display the "privacy policy", which would be a problem for users in some countries.
from twentynineteen.
I think one definite pro for this way is simplicity (decisions above options :-)).
I am not sure about what the Impressum is, but isn't that something for a widget area in the footer? #14
Anyway, I just noticed that this is already in the theme, in about the same way... :-)
from twentynineteen.
Regardless, it needs a check for if the function exists, or is there somewhere checking for minimum version (and graceful non-activation)? I thought default themes couldn't do that, but I guess Twenty Seventeen has a 4.7 minimum.
from twentynineteen.
Impressum is similar to privacy policy, just different content (contact details and responsibility
of the website and stuff like that and a "must have" in germany (and I think in EU as well)) and should always be displayed next to privacy policy.
I like the new core function for GDPR and it would be great, that a defined GDPR-Page get displayed automatically in the theme. We just should make it easy for users, to add more pages if they need to, to match their country requirements without the knowledge of CSS, HTML or even PHP.
-
So maybe an Setting Option to add more pages next to it
-
or the option to deactivate those function and the possibility to define an own footer Menu?
from twentynineteen.
There already is a link to the privacy policy page in footer.php.
from twentynineteen.
There already is a link to the privacy policy page in footer.php.
Agreed βΒ just for clarification's sake: are we discussing something in addition to what's here already?
https://github.com/WordPress/twentynineteen/blob/master/footer.php#L21
from twentynineteen.
Ok, thanks for the clarification.
The only thing is that the check on the existence of the privacy policy function might be superfluous if the theme would check the WP version (for only 5+).
I'm note sure about this, but I do think the current solution works fine as is. We can pursue this in a separate issue if we'd like. Thanks, all!
from twentynineteen.
Related Issues (20)
- Video embed issues HOT 1
- Keyboard menu showing with touch menu on Mobile HOT 2
- Packagist and NPMJS HOT 1
- Editor vs. Preview/Front end alignments of blocks in columns. HOT 1
- Baskerville Old Face is a poor font choice for Windows HOT 4
- Metaboxes are overlapping the Block editor HOT 5
- Convert references to 'classic editor' and 'block editor' to proper nouns HOT 2
- Heading not aligned with the rest of the content HOT 3
- Two column block: sub-blocks not aligned in mobile view HOT 1
- No version number in doc block HOT 3
- Update version numbers for 5.0.2 release to sync with Core SVN
- .button class doesn't override link color HOT 1
- Search widget input field width is too narrow on wide screens HOT 1
- alignwide in editor causes horizontal scroll HOT 1
- Firefox Dropcap margin mismatch [Proposed Solution] HOT 2
- touch-keyboard-navigation.js conflicts with some lightboxes HOT 1
- After a word written in italic, inverted commas appear wrongly HOT 1
- Deleting all sass files HOT 2
- touch-keyboard-navigation.js breaks scrolling past links on touch HOT 2
- where are the SCSS files for the style-RTL.css HOT 3
Recommend Projects
-
React
A declarative, efficient, and flexible JavaScript library for building user interfaces.
-
Vue.js
π Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.
-
Typescript
TypeScript is a superset of JavaScript that compiles to clean JavaScript output.
-
TensorFlow
An Open Source Machine Learning Framework for Everyone
-
Django
The Web framework for perfectionists with deadlines.
-
Laravel
A PHP framework for web artisans
-
D3
Bring data to life with SVG, Canvas and HTML. πππ
-
Recommend Topics
-
javascript
JavaScript (JS) is a lightweight interpreted programming language with first-class functions.
-
web
Some thing interesting about web. New door for the world.
-
server
A server is a program made to process requests and deliver data to clients.
-
Machine learning
Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.
-
Visualization
Some thing interesting about visualization, use data art
-
Game
Some thing interesting about game, make everyone happy.
Recommend Org
-
Facebook
We are working to build community through open source technology. NB: members must have two-factor auth.
-
Microsoft
Open source projects and samples from Microsoft.
-
Google
Google β€οΈ Open Source for everyone.
-
Alibaba
Alibaba Open Source for everyone
-
D3
Data-Driven Documents codes.
-
Tencent
China tencent open source team.
from twentynineteen.