ElkArte TinyPortal Layout & Design

[McFly]

For ELK1

First, I removed the "clearthefloat" paragraphs. This did not require any change to the CSS (for now).

However, I noticed the following oddity. For the DIVs "tpleftbarheader" and "tprightbarheader" the margin widths are taken inline from the settings. But for "tpleftbarheader" only style="" is displayed, for "tprightbarheader" the specification is missing completely. However, the widths are displayed as "inline" in the developer (firefox) and are also displayed correctly. 

[@rjen]

That is due to the collapse function..

This line comes from SMF version, but I assume it is there in Elk too:

<div id="tpleftbarHeader" style="' , in_array('tpleftbarHeader',$context['tp_panels']) && $context['TPortal']['showcollapse']==1 ? 'display: none;' : '' , '">

If the collapse is selected it will toggle to style="display: none;"
I suppose the style element could have been but into the condition as well..

This would fix that empty style I guess..

<div id="tpleftbarHeader" ' , in_array('tpleftbarHeader',$context['tp_panels']) && $context['TPortal']['showcollapse']==1 ? 'style="display: none;"' : '' , '>

[McFly]

I know what you mean. You mean that the missing style expression is because it is the collapse function. But this function sets the two or three DIVs following "tp-left/rightblock_body" to "display: none".

But I meant the following: For <div id="tpleftbarHeader" style=""> there should be a "margin-right: 4px" inside the "style" specification, for <div id="tprightbarHeader" > the "style" specification is completely missing. However, the margins are rendered correctly and also appear in the browser's "inline" enumeration.

[McFly]

ELK1

Removed two classes no longer used for input buttons (button, button_submit), also adopted the "submitbutton" class for most save button DIVs. A few typos corrected.

[@rjen]

ELK1

Removed two classes no longer used for input buttons (button, button_submit), also adopted the "submitbutton" class for most save button DIVs. A few typos corrected.

This is another one of these things that I will not (yet) do in SMF version: because we are supporting both SMF2.0 (using class="button_submit") and SMF2.1 (class="button"). This is the kind of stuff I can pickup for the SMF version when we drop 2.0 support...

Good that you do it now for the Elk version though...

[tino]

ELK1

Removed two classes no longer used for input buttons (button, button_submit), also adopted the "submitbutton" class for most save button DIVs. A few typos corrected.

Thanks for these.

Are you comfortable using Git/GitHub? It's alot easier to track these changes if you can submit PR's against the main repo, to then be merged.

[McFly]

...

Are you comfortable using Git/GitHub? It's alot easier to track these changes if you can submit PR's against the main repo, to then be merged.

I can give that a chance. I'll have to read up on it first and get to know it.

[McFly]

Now the first extension at the CSS. For this, the menus in the admin area are now built according to ELK model.

I noticed that the menu under "Block access" is missing. Feature? 

[McFly]

Just a small revision to the CSS classes "blockbody" and "h2.category_header ...", as "blockbody" fabricates an unsightly border when rendering a block without title and border styles.

[McFly]

Again, a few typos weeded out. Found some more "submitbutton" and changed quite a few style="float: left/right", "clear: whatever" and "overflow: auto/hidden" in existing classes, and introduced new class for the bullets.

[McFly]

It's really not good when I have too much spare time ... 

I have revised the category list again and also adapted the article list in the same way. No more tables, no inline styles and similar to ELK.

[tino]

Thanks! I'll push these to github later this week.

Are all the changes included in the latest zip file? Or do I need to apply them in order.

[McFly]

...

Are all the changes included in the latest zip file? Or do I need to apply them in order.

The latest zip file always contains the previous changes.

[McFly]

There is more to come! Have now finished work ...

Again a few typos. For this I have - with one exception - all replaced by the class strong. But for this to work, a bug in ELK's index.css has to be fixed first. Around the line 750 a dot has been forgotten there before the class.

[tino]

But for this to work, a bug in ELK's index.css has to be fixed first. Around the line 750 a dot has been forgotten there before the class.

We can't rely on that being fixed, so will have to have our own tp override. Or similar.

[McFly]

...

We can't rely on that being fixed, so will have to have our own tp override. Or similar.

For this I have just added a class in the tp-styles. The file will be available later.

By the way, I also just found the thing with the inline styles of "tp-left/rightbarheader"! They are hidden in the HTML header as scripts. Are defined in "Subs.php" from line 75.

EDIT:

Cleaned out the CSS itself a bit and added the "strong" class.

[@rjen]

Just a warning (have not checked the css yet) but if you are using very common class names (such as strong or float left) it is advisable to create distinct class names , such as tp_strong or tp_floatleft to prevent issues with conflicting class names in themes or other mods ...

[McFly]

Just a warning (have not checked the css yet) but if you are using very common class names (such as strong or float left) ...

I definitely spent too much time in front of the monitor yesterday! Or: when you can't see the forest for the trees. 

Good that I have again slept a night over it. It is of course NOT an error in the CSS of ELK. I will revise the files again.

EDIT: Attached zip file replaced.

[McFly]

Just a small revision of the category and article list, CSS adjustment.

[tino]

Thanks, I'll get this checked in to git today hopefully!

[McFly]

Reworked article in cat page (tableless) and CSS.

[McFly]

The next bunch of changes. The descriptive texts are now also all implemented as paragraphs below the header.

[tino]

Thanks for all these!

All your changes should be in this commit https://github.com/tinoest/ElkTinyPortal/commit/ff9d200c228dbaf358124dbcd22803163269267f I'll push it to the main repo this weekend.

[McFly]

Thanks for all these!

All your changes should be in this commit https://github.com/tinoest/ElkTinyPortal/commit/ff9d200c228dbaf358124dbcd22803163269267f I'll push it to the main repo this weekend.

Then I will wait for that and after that I will archive my current WIP folder and open a new WIP with the repository up to date.

[tino]

Thanks for all these!

All your changes should be in this commit https://github.com/tinoest/ElkTinyPortal/commit/ff9d200c228dbaf358124dbcd22803163269267f I'll push it to the main repo this weekend.

Then I will wait for that and after that I will archive my current WIP folder and open a new WIP with the repository up to date.

Should be available here https://github.com/Tinyportal/ElkTP it's the master branch

I have also synced the development branch for Elkarte 2 with the changes you made to keep them in sync.