Skip to content

Conversation

kp-org
Copy link

@kp-org kp-org commented Aug 29, 2018

This is a simple change to hide the Page Id at the upper right corner of the text area of the default template. By default it is hidden, but could be made visible via config option (plugin config). Additional the file main.php contains a similar change to hide the "last modified" info, but not configurable via the config plugin (yet).

@phy25
Copy link
Collaborator

phy25 commented Aug 30, 2018

Maybe these can be a template-dependent options. Also I doubt whether the use of HTML comments is efficient.

@splitbrain
Copy link
Collaborator

This is template specific and thus should not be in the core configuration. In addition I do believe that these are sensible features of the DokuWiki experience and should not be disabled. The ID of a page is important information that is used for linking. It should not be hidden from the user. The same goes for information about who changed a page.
If your use case is different from a traditional wiki, use a different or customized template.

@kp-org
Copy link
Author

kp-org commented Sep 7, 2018

These are a visual effects only and can be switched on or off. No info will be thrown away. And e.g. the page id is duplicate info as it is in the address bar. It doesn't change the behavior of dokuwiki by default. Question the use of the config plugin leads to question if the trace/breadcrumbs option isn't template specific too. Btw. it works for me for years. But your decision, I just wanted to help and give some ideas.

@Michaelsy
Copy link
Contributor

Michaelsy commented Oct 2, 2018

@kp-org: Out of the box it is very simple to hide these two elements. Take this CSS rules to your conf/userstyle.css and all is gone:

.docInfo {
    display: none;
}

.pageId {
    display: none;
}

You can take a look at a live demo.

Just my 2 cents - Michael Sy.

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

Successfully merging this pull request may close these issues.

4 participants