TP-Docs
HTML5 Icon HTML5 Icon HTML5 Icon
TP on Social Media

Recent

Welcome to TinyPortal. Please login or sign up.

July 20, 2024, 04:45:50 PM

Login with username, password and session length
Members
Stats
  • Total Posts: 195,333
  • Total Topics: 21,233
  • Online today: 133
  • Online ever: 3,540 (September 03, 2022, 01:38:54 AM)
Users Online
  • Users: 0
  • Guests: 67
  • Total: 67

ElkArte TinyPortal Layout & Design

Started by McFly, March 22, 2022, 11:10:37 AM

Previous topic - Next topic

0 Members and 1 Guest are viewing this topic.

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.  ???
"I'm not going to censor myself to comfort your ignorance."

Testsites (may be online or not ;-) )
ElkArte 1.1.x & TP on Apache2.4.48, PHP 8.1.2 FPM/FastCGI, MariaDB 10.5.12

@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;"' : '' , '>
Running Latest TP on SMF2.1 at: www.fjr-club.nl

McFly

#2
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.
"I'm not going to censor myself to comfort your ignorance."

Testsites (may be online or not ;-) )
ElkArte 1.1.x & TP on Apache2.4.48, PHP 8.1.2 FPM/FastCGI, MariaDB 10.5.12

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.
"I'm not going to censor myself to comfort your ignorance."

Testsites (may be online or not ;-) )
ElkArte 1.1.x & TP on Apache2.4.48, PHP 8.1.2 FPM/FastCGI, MariaDB 10.5.12

@rjen

Quote from: McFly on March 24, 2022, 09:33:46 AM
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...
Running Latest TP on SMF2.1 at: www.fjr-club.nl

tino

Quote from: McFly on March 24, 2022, 09:33:46 AM
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

Quote from: tino on March 24, 2022, 01:29:11 PM
...

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.
"I'm not going to censor myself to comfort your ignorance."

Testsites (may be online or not ;-) )
ElkArte 1.1.x & TP on Apache2.4.48, PHP 8.1.2 FPM/FastCGI, MariaDB 10.5.12

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?  ;)
"I'm not going to censor myself to comfort your ignorance."

Testsites (may be online or not ;-) )
ElkArte 1.1.x & TP on Apache2.4.48, PHP 8.1.2 FPM/FastCGI, MariaDB 10.5.12

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.
"I'm not going to censor myself to comfort your ignorance."

Testsites (may be online or not ;-) )
ElkArte 1.1.x & TP on Apache2.4.48, PHP 8.1.2 FPM/FastCGI, MariaDB 10.5.12

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.
"I'm not going to censor myself to comfort your ignorance."

Testsites (may be online or not ;-) )
ElkArte 1.1.x & TP on Apache2.4.48, PHP 8.1.2 FPM/FastCGI, MariaDB 10.5.12