[NODEBB] CSS Style Sheets SelectBox

[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 ?

[DownPW]

off course @phenomlab
I search to reducs space between Icon search bar and notification icon

[phenomlab]

@downpw Try

#switcher {
    margin-right: -15px;
    margin-left: -15px;
}

[DownPW]

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

#switcher {
margin-right: -15px;
margin-left: -15px;
}

Yes very Good !!!
Like this The button is well : Thanks

[phenomlab]

@downpw

Right. I moved the custom HTML to a HTML based global widget, and that works Give it a try

Also, not sure why you are using the html class for the background images. You should use body. I just tried it, and it works fine

[phenomlab]

@DownPW some additional advice. The images you are calling remotely should be hosted on your own server. This will prevent unnecessary remote calls, and will also load faster. I’m seeing delay times of up to a second before the wallpaper appears