Skin talk:MonoBook

Jump to navigation Jump to search

About this board

this->data['loggedin'] not working when using wfLoadSkin()

1
68.168.100.132 (talkcontribs)

Hi, I have this snippet of code in mi skins/MonoBook.php file:

<?php if(!$this->data['loggedin']) { ?>

     <style>
       .editsection { display: none; }
       .editornote { display: none; } 
       div#p-editors {display: none; }
     </style>
<?php } ?>


Seems not to be working.

Someone suggested it probably have something to do with require_once() not being used to load it, but I don't think that is the case.

Any suggestions?

Reply to "this->data['loggedin'] not working when using wfLoadSkin()"
Be..anyone (talkcontribs)

The visible content of a page under the monobook skin consists of (left to right) the logo and a toolbar (language, user, talk, preferences, etc.) in the first row, the logo and the tabs in the second row, the logo and the header in the third row, and so on. The logo is linked to the main page. On commons I translated a template to German, and to test the effect I switched the language from en-GB to de-DE, screenshots:

As you can see I was trapped with "Deutsch" (German), because clicking on it was handled by the logo link, instead of whatever is supposed to switch languages. The German version of the toolbar is far wider than the en-GB toolbar, despite of the verbose "British English" vs. "Deutsch" button text. To reset the language I had to disable a tool in preferences temporarily, after that I could click on "Deutsch" to switch the language, and reenable the tool.

Suggestion: The logo could be arranged as area, and the top of this area should not be linked, giving any overflows from the toolbar a chance to do what they are supposed to do. Be..anyone (talk) 14:17, 12 November 2014 (UTC)

Reply to "Layout issue"
There are no older topics