OWASP / www--site-theme

Contains owasp site theme specific items (headers, footers, json, menus)
9 stars 27 forks source link

can we not mess around with master theme if its linked to all projects #17

Closed anantshri closed 4 years ago

anantshri commented 4 years ago

can we not mess around with master theme atleast the master node of it as its linked with all projects. currently it has rendered all the pages in a not so good state. sidebars are all gone below the text.

Current render due to theme errors

owasp_bhopal_defunc_render

What it should be

OWASP_bhopal_should_be_render

bkimminich commented 4 years ago

Looks a lot like a messed-up version of the small-screen layout to me... On my laptop the page looks fine and when I reduce window size goes into smal-screen layout which also looks okay - except for the custom grey box not having bottom padding and proper width:

image

anantshri commented 4 years ago

this mess was made due to changes done on master theme as this theme is live referred by every subsite any changes in master are immediately reflected in the other sites.

later commit assuming https://github.com/OWASP/www--site-theme/commit/66d7626e25fc5bd32d47ddc6886953a5adf38f7d fixed the layout back to normal.