[NODEBB] CSS Style Sheets SelectBox

[DownPW]

First, I wanted to thank you for your great job

I have a lot of questions !

• I don’t see a modification on client.css our my custom css file?

• Just JS for all that ? No modified other file?

• How do you fix the yesterday problem for CSS on chat? (How do you include default client.css with my custom CSS files?)

• My custom CSS on ACP/custom CSS is just for default Theme (yes If I understood correctly) ?

• Can I disable all my customcss code on body.lightout variables on ACP/Custom CSS, (used for night mode) ?

• Can I add other theme? --> Just add following if I’m not mistaken?

---

And I have other cosmetic request :

• How can I fix this little problem with the spacing between the search icon–the theme switcher icon–notification icon?
  

• I have not mouse over color on theme switcher icon like other icons on navbar?
  

It’s the same for all themes:

• When I select a theme, I have this, how to fix these :
  

+On Mobile, can we delete these 2 lines

• On mobile again, my wallpaper doesn’t apply (It was the case before, like default theme on ACP/custom CSS, the css code is the same)

• I have this on console, normal ?

---

For resume, If I want to do all reproduce all that on production, If I understood correctly :

• Disable Night Mode. Rebuild NodeBB
• Copy my CSS file on prod server at the same repertories (or other if I change the JS code)
• Copy Paste your Custom JS Code on ACP/Custom JS
• Recover with Broswer Console the code ?v=XXXXX used by noidebb instance
• Modify URL of Stylesheets with my css custom files path and ?v=XXXXX on your JS Code
• Rebuild NodeBB

[phenomlab]

@downpw said in [NODEBB] CSS Style Sheets SelectBox:

I don’t see a modification on client.css our my custom css file?

That’s correct. There is no need to modify any of the custom CSS files

@downpw said in [NODEBB] CSS Style Sheets SelectBox:

Just JS for all that ? No modified other file?

Yes - jQuery in fact. You no longer require the widget HTML either as it’s now all wrapped in a custom function.

@downpw said in [NODEBB] CSS Style Sheets SelectBox:

How do you fix the yesterday problem for CSS on chat? (How do you include default client.css with my custom CSS files?)

The default CSS is loaded by NodeBB, so will always be there. The custom function I wrote uses .append to simply add the custom CSS to what is already there

My custom CSS on ACP/custom CSS is just for default Theme (yes If I understood correctly) ?

No. That’s a global setting. No matter what is loaded by the custom function, these additional styles will always be added inline.

@downpw said in [NODEBB] CSS Style Sheets SelectBox:

Can I disable all my customcss code on body.lightout variables on ACP/Custom CSS , (used for night mode) ?

Yes

@downpw said in [NODEBB] CSS Style Sheets SelectBox:

Can I add other theme? --> Just add following if I’m not mistaken?

Yes, you can. You just need to copy the same format with different theme names, ID’s, and of course, the rel link.

@downpw said in [NODEBB] CSS Style Sheets SelectBox:

How can I fix this little problem with the spacing between the search icon–the theme switcher icon–notification icon?

Looks like it’s already fixed ?

@downpw said in [NODEBB] CSS Style Sheets SelectBox:

I have not mouse over color on theme switcher icon like other icons on navbar?

You can instate that using class .dropdown-toggle:hover

@downpw said in [NODEBB] CSS Style Sheets SelectBox:

When I select a theme, I have this, how to fix these :

This also seems to be fixed ?

@downpw said in [NODEBB] CSS Style Sheets SelectBox:

On Mobile, can we delete these 2 lines

That is controlled by CSS class .navbar-form using box-shadow: inset 0 1px 0 rgba(255,255,255,.1),0 1px 0 rgba(255,255,255,.1); so you’d need to either modify that, or remove it using box-shadow: none;

@downpw said in [NODEBB] CSS Style Sheets SelectBox:

On mobile again, my wallpaper doesn’t apply (It was the case before, like default theme on ACP/custom CSS , the css code is the same)

I noticed this also, but now note that the wallpaper seems to be the same even on desktop despite the color being switched. Is this intended, as this was not the case yesterday ?

@downpw said in [NODEBB] CSS Style Sheets SelectBox:

I have this on console, normal ?

Sorry - I can’t see any screenshot ?

@downpw said in [NODEBB] CSS Style Sheets SelectBox:

For resume, If I want to do all reproduce all that on production, If I understood correctly :

Disable Night Mode. Rebuild NodeBB
Copy my CSS file on prod server at the same repertories (or other if I change the JS code)
Copy Paste your Custom JS Code on ACP/Custom JS
Recover with Broswer Console the code ?v=XXXXX used by noidebb instance
Modify URL of Stylesheets with my css custom files path and ?v=XXXXX on your JS Code
Rebuild NodeBB

Yes, that’s pretty much it

[DownPW]

@phenomlab

@phenomlab said in [NODEBB] CSS Style Sheets SelectBox:

Looks like it’s already fixed ?

Yes, I have fixed with this :

.notifications.dropdown.xs {
    margin-left: -18px;
    margin-right: -18px;
    margin-top: 6.2px;
    list-style: none;
}

---

@downpw said in [NODEBB] CSS Style Sheets SelectBox:

When I select a theme, I have this, how to fix these :

This also seems to be fixed ?

No, not fixed
I have test several things with no success:

---

@downpw said in [NODEBB] CSS Style Sheets SelectBox:

On mobile again, my wallpaper doesn’t apply (It was the case before, like default theme on ACP/custom CSS , the css code is the same)

I noticed this also, but now note that the wallpaper seems to be the same even on desktop despite the color being switched. Is this intended, as this was not the case yesterday ?

No change, It’s because I have Custom Header code on ACP (Your Reading Meter Bar) and he bug the css code of others theme except default theme.

Any code adding on ACP/Custom Header like a div for example crashes CSS themes :

If i disable code in ACP/Custom Header. The CSS custom themes is OK except wallpaper on Mobile

---

@downpw said in [NODEBB] CSS Style Sheets SelectBox:

I have this on console, normal ?

Sorry - I can’t see any screenshot ?

Sorry, I can’t edit the post yesterday:

---

@phenomlab said in [NODEBB] CSS Style Sheets SelectBox:

@downpw said in [NODEBB] CSS Style Sheets SelectBox:

I have not mouse over color on theme switcher icon like other icons on navbar?

You can instate that using class .dropdown-toggle:hover

I have test with that on ACP = doesn’t work :

.dropdown-toggle:hover {
    background: #3C3C3C;
}

But Ok with this code but the background doesn’t occupy the entire height of the navbar like other navbar icons :

.notifications.dropdown.xs .dropdown-toggle:hover {
    background: #3C3C3C;
}

Here, this is OK. Iwould like this result:

[phenomlab]

@downpw Ok. Let me have a look

[phenomlab]

@DownPW I’ve set this custom CSS in your ACP

// PLEASE DO NOT DELETE THIS

.header #theme_dropdown {
    padding: 9px 15px;
}
.header #theme_dropdown {
    padding: 9px 15px;
        padding-top: 9px;
        padding-bottom: 9px;
    margin-top: -6px;
    padding-top: 14px;
    padding-bottom: 13px;
}

This is the basis which should make it work, but you’ll need to “fiddle” with it to get it how you need.

I also changed the jQuery code slightly, as I realized there was a conflict with another ID, which I’ve fixed.

EDIT: It’s this line

<label for="user-control-list-check" class="dropdown-toggle" data-toggle="dropdown" id="***theme***_dropdown" title="" role="button" data-original-title="Theme" aria-expanded="false"> \

[DownPW]

OK Thanks.
You have change id="user_dropdown with id=“theme_dropdown”

Can you will see for others problem after?

[phenomlab]

@downpw I’m running low on free time now sadly, but more then happy to get this resolved for you tomorrow or later in the week of that works for you.

The issue here is that I’ve reused some of the existing code from NodeBB and that may be causing some issues. Don’t worry. We’ll get there

[DownPW]

@phenomlab said in [NODEBB] CSS Style Sheets SelectBox:

@downpw I’m running low on free time now sadly, but more then happy to get this resolved for you tomorrow or later in the week of that works for you.
The issue here is that I’ve reused some of the existing code from NodeBB and that may be causing some issues. Don’t worry. We’ll get there

OK Thanks sir
I really hope ! You are a super hero !

I also hope that when done this will work on future nodeBB updates/upgrade.
PM me when you have the time for that I turn on the VM

[phenomlab]

@downpw no issues. It’ll work across upgrades as it’s a standard function and doesn’t rely on any NodeBB components. If you can just ensure that the VM is available around the same time each day, that would also help.

[DownPW]

@phenomlab said in [NODEBB] CSS Style Sheets SelectBox:

If you can just ensure that the VM is available around the same time each day, that would also help.

Ok no problem Sir

[phenomlab]

@downpw Right. I think I have this working the way you’d like

This uses the below CSS I have placed into your ACP. It should not be deleted

// PLEASE DO NOT DELETE THIS

.header #theme_dropdown {
    padding: 9px 15px;
    padding-top: 9px;
    padding-bottom: 9px;
    margin-top: -4px;
    padding-top: 14px;
    padding-bottom: 16px;
}
#switcher{
    position: absolute;
    right: 33%;
}

The :hover class is always going to look like the below

The reason for this is because you have it hard coded in the ACP CSS as below

/*VIOLENCE: Couleur du background au survol souris */
.navbar-default .navbar-nav>li>label:hover {
    background: #555555;
}

You should remove this, and set it based on the color you want in each external CSS file. This way, it will display correctly.

The slight caveat with this is that it has to use absolute positioning meaning that if you resize the browser, the theme switcher icon will use a percentage to work out where it needs to be, and won’t be governed by the <ul><li> it sits inside. This is the only way to get the effect you are looking for.

[DownPW]

@phenomlab said in [NODEBB] CSS Style Sheets SelectBox:

@downpw Right. I think I have this working the way you’d like

This uses the below CSS I have placed into your ACP. It should not be deleted

// PLEASE DO NOT DELETE THIS

.header #theme_dropdown {
    padding: 9px 15px;
    padding-top: 9px;
    padding-bottom: 9px;
    margin-top: -4px;
    padding-top: 14px;
    padding-bottom: 16px;
}
#switcher{
    position: absolute;
    right: 33%;
}

The :hover class is always going to look like the below

The reason for this is because you have it hard coded in the ACP CSS as below

/*VIOLENCE: Couleur du background au survol souris */
.navbar-default .navbar-nav>li>label:hover {
    background: #555555;
}

You should remove this, and set it based on the color you want in each external CSS file. This way, it will display correctly.

The slight caveat with this is that it has to use absolute positioning meaning that if you resize the browser, the theme switcher icon will use a percentage to work out where it needs to be, and won’t be governed by the <ul><li> it sits inside. This is the only way to get the effect you are looking for.

Hmmm, it’s problematic. It’s not very aesthetic.
I am amazed because it works very well on your site or with night mode.

I see red background too on fa -faw ???

[phenomlab]

@downpw said in [NODEBB] CSS Style Sheets SelectBox:

Hmmm, it’s problematic. It’s not very aesthetic.
I am amazed because it works very well on your site or with night mode.

The theme switcher you have doesn’t work the same way as the night mode plugin. What you have is a dropdown which is not the same as the toggle selection hence it will behave differently.

What isn’t aesthetic exactly ?

@downpw said in [NODEBB] CSS Style Sheets SelectBox:

I see red background too on fa -faw ???

Yes, that comes from the below class on line 415 of your CSS in the ACP

/*VIOLENCE: Couleur du background de l'icone de recherche au survol souris */  
.navbar-default .btn-link:hover {
    background: red;
}

[phenomlab]

@DownPW another way…

Remove

#switcher{
    position: absolute;
    right: 33%;
}

Add

.navbar-default {
    height: 50px;
}

This way, it’s aesthetic…I just set the changes. Let me know what you think.

[DownPW]

@phenomlab

It’s better.

I have just to set space between icon

For the red, i have disable CSS on line number 415, it’s not that

[phenomlab]

@downpw said in [NODEBB] CSS Style Sheets SelectBox:

For the red, i have disable CSS on line number 415, it’s not that

Odd. This will stop it

#switcher .btn-link:hover {
    background: none !important;
}

[DownPW]

Yeah It’s good

[phenomlab]

@downpw Great. I think we’re done, yes ?

[DownPW]

Hmm I search to reducs space between Icon search bar and notification icon

[phenomlab]

@downpw Can you provide a screenshot to explain what you are looking for ?