chat list navbar

[DownPW]

@phenomlab

Yeah good for “Mark all as read” but not All chats button:

@DownPW said in chat list navbar:

yeah great…

Can you test All chats button hover ?
Because .btn-group>.btn:last-child:not(:first-child), .btn-group>.dropdown-toggle:not(:first-child):hover not work

[phenomlab]

@DownPW said in chat list navbar:

Yeah good for “Mark all as read” but not All chats button:

Yes, sadly, that element is in use in several other places. You could target it directly, but it will also mean changing it in multiple places which won’t be the desired effect. The most effective way would be to target a div ID, but you’d need something like jQuery to target that specific element and add an ID to it.

[DownPW]

@phenomlab

hmm hard for just an effect.
It’s a problem for my dark themes

In sudonix I see this :

and Hover:

I think it’s a better way, how achieve this effect (with different color) ?

[phenomlab]

@DownPW Ah, yes, I remember this one now - it was a nightmare to target, and I landed up using the pseudo class which unfortunately does include the border, but it’s subtle enough to still look good

Try this

.btn-group.btn-group-justified:hover {
    /* Your css here */
}

[DownPW]

I resolve with this

.btn-group>.btn:last-child:not(:first-child), .btn-group>.dropdown-toggle:not(:first-child) {
    color: #ccc;
}

a.btn.btn-secondary:hover {
  color: #ccc;
  background: #212427;
}

.caret {
    color: #000;
}

a.btn.btn-secondary.mark-all-read {
	color: #ccc !important;
}

a.btn.btn-secondary.mark-all-read:hover {
  background: #212427;
	color: #ccc !important;
}

[DownPW]

@phenomlab

I have a problem on my themes.

Example I Have this on my css files :

.btn-group>.btn:last-child:not(:first-child), .btn-group>.dropdown-toggle:not(:first-child) {
    color: var(--first-color) !important;
}

But this code is not taken into account

if i change in dev console (in live) it’s ok

Can you help me with that ?

[DownPW]

Maybe a problem with theme switcher & nodeBB 3.2.1 ?

[phenomlab]

@DownPW no, this works independently of NodeBB as it uses jQuery and CSS. The issue you have is the below

var(--first-color)

In CSS, var is a variable defined at root level and probably doesn’t exist in your CSS file. Essentially, it’s LESS. If you change the line so it looks (for example) like the below, it will probably then work

#ffffff;

The newer version of the theme switcher uses LESS variables as it’s more efficient but if these don’t exist then it looks like it’s being ignored when I’m fact it just returns a null value.

[DownPW]

yes i know for var all the theme is coded with var.

It’s the same things with #XXXXXX

[phenomlab]

@DownPW this sounds more like a caching issue to me. Do you get the same thing in an incognito session ?

[DownPW]

@phenomlab said in chat list navbar:

@DownPW this sounds more like a caching issue to me. Do you get the same thing in an incognito session ?

yes

[DownPW]

I just installed firefox and it’s the same so for me it’s not a cache problem

Edit @phenomlab

For example, in fortifem theme, the code is correctly interpreted but not on Nightprowler theme but the code is the same !

Fortifem its’ ok :
https://i.imgur.com/Y85sVBi.gif

Night prowler is not Ok withe the same code as Fortifem (variable) :

https://i.imgur.com/tWus88b.gif

All the rest of theme is OK

[phenomlab]

@DownPW I’ll have a look at this tomorrow. Sounds like you have values being overridden somewhere

[DownPW]

@phenomlab

I got better and I think I understand a little more the problem

I declared the code in the ACP (it was not the case yesterday - just in the CSS file yesterday) and it applies. Normal

On the other hand, it only applies the code of the ACP and not that of the CSS file.

[DownPW]

EDIT @phenomlab :

I just tested by deleting the entire CSS file except the button code and it is taken into account.

There is therefore a CSS variable which prevents the execution of the code, but from there to know which one

[DownPW]

EDIT2:

I find it…

[DownPW]

hi @phenomlab

I have this little avatar mismatch in the chat menu (just in Full mode) but only some of my themes (exemple: Daystalker) and I can’t find what’s wrong.

An idear mark?

[DownPW]

Hmm I find it the problem is that :

.chats-list>li {
  padding: 0.5em;
  height: 80px; 
}

[phenomlab]

@DownPW yes, you may need to be more specific with the target. Let me know

[DownPW]

No no it’s ok @phenomlab
I just comment the 2 lines mentionned aboves