The currently selected checkin now has an ugly background that seems like it is some hardcoded value and not from the skin, see https://objfw.nil.im/info/4666442ec26329cc for example.

https://facepalm.synology.me/fossil/ardoise/doc/trunk/www/screenshots/checkin.png shows how it is supposed to look.

If I did the bisecting correctly, looks like this commit broke the view.

The skin's received several updates recently. Have you either merged those changes into your local skin, or reset your skin to the stock state?

When you select a non-default skin, you're making a copy of the skin at the time of selection which then stays static forevermore, until you edit it again. Your local copy doesn't auto-upgrade when the stock version of the skin changes, because it might have local edits that would conflict.

It's why the Fossil skin editor has the diffing facilities: so you can quickly see what's changed and decide how much of it you want to adopt.

The skin's received several updates recently. Have you either merged those changes into your local skin, or reset your skin to the stock state?

Setting up a new Fossil repo and changing from the default skin to Ardoise in either draft1, or making it the default results in the same look that js linked to.

Are there changes someone would need to make on a brand new repo in order for Ardoise to look correctly?

I'm aware, and it's the first thing I suspected and tried. But resetting the skin makes absolutely no differnece.

The currently selected checkin now has an ugly background that seems like it is some hardcoded value and not from the skin, see https://objfw.nil.im/info/4666442ec26329cc for example.

Hmmmm. i'm curious how this slipped by me in the testing with the version Sean points to and in Andy G's recent touchups of that skin (which are far-derived from that version).

Here's a workaround which is working for me via the browser dev tools and i'll take a look at updating the skin to account for this: in the skin editor, add this to the bottom:

.timelineSelected {
  background-color: initial
}

This is now fixed in the trunk, just a variant of what i suggested in my last post:

Index: skins/ardoise/css.txt
==================================================================
--- skins/ardoise/css.txt
+++ skins/ardoise/css.txt
@@ -9,10 +9,11 @@
 table.login_out,
 table.login_out td,
 tr.timelineCurrent,
 tr.timelineCurrent td.timelineTableCell,
 tr.timelineSelected {
+  background-color: initial;
   border: 0
 }
 ol,
 p,
 ul {

You can simply make that change in the skin editor, rather than updating your fossil version.

Thank you for the report!

Thx, this comes very close to it, but there is still some slight shadow to the left of it that wasn't there before: https://objfw.nil.im/info/f79f04f882c43fa6

Thx, this comes very close to it, but there is still some slight shadow to the left of it that wasn't there before: https://objfw.nil.im/info/f79f04f882c43fa6

i'm not going to chase that down because i consider it to be a personal preference rather than a problem and i actually like it better that way (there's no accounting for taste, after all). (Note that in 2.13 more Ardoise differences will appear, courtesy of Andy Goth.)

That said, if you will find and post the CSS which eliminates that shadow i'll integrate it.

box-shadow: none in .timelineSelected does the trick. Can you add that as well, please?

Done! A similar shadow is (still) on .timelineCurrent, just FYI.

Where would this show up? I didn't see any shadows anywhere anymore after changing .timelineSelected.

Where would this show up? I didn't see any shadows anywhere anymore after changing .timelineSelected.

This was the one-line change you requested to the Ardoise skin:

https://fossil-scm.org/fossil/info/3af6e7ceb4423169

It will affect any elements with the timelineSelected class, like you requested above:

box-shadow: none in .timelineSelected does the trick. Can you add that as well, please?

I think he is asking where .timelinecurrent's box shadow shows up.

Yep, exactly that.

That does not seem to contain the other change, though?

But what I really meant is where I would still see that shadow, because I haven't found it anywhere.

That does not seem to contain the other change, though?

i'm not sure which other change you're referring to unless you mean the unisghtly background color inheritance, which was fixed in trunk near the top of this thread.

As to:

I think he is asking where .timelinecurrent's box shadow shows up.

Sorry, i confused timelineSelected and timelineCurrent. If i'm understanding the C code correctly, timelineCurrent is the current version of a checkout, as opposed to the "currently selected" version. Apparently it only appears when running the UI from a checkout.

If you're interested in the Ardoise skin, have a look at the andygoth-ardoise-tweaks branch and feel free to contribute. It's not 100% the way I want it yet, just haven't had time to tinker.