concise.css/src/_atgrid.scss [container]

[ghost]

No description provided.

[ghost]

James,

The previous "issue" I opened and you accepted on this topic has a problem. Sorry.

If at the end of a container there is something similar to a dropdown menu with hidden items that extend beyond the container, a double scroll bar is displayed. See attached screenshot. The screenshots show Firefox, Chrome and Safari under OS-X.

I stumbled across a very simple and Concise solution.

1. Remove the "overflow: auto" we added to _atgrid.scss
2. In concise.css/src/_base.scss
   :root {
   -- background-color: getColor(background, body);

++ body {
++ background-color: getColor(background, body);
++ }

I am uncertain why this fix works, it makes no sense to me, but it does.

I believe this change is safe for the current version of Concise CSS.

[ghost]

Hi @fredhall

I also noticed that changing the background color from :root to body solved the problem, but because your solution also worked for me, I used that one, I didn't know it also created another scrollbar.

So I’ll add this change, but I’m not quite sure if it’s safe for a patch version because some users might be doing unexpected things so we can’t assume. What I’m going to do, is to remove the overflow: auto and change the background rule in a next major version.

But don’t worry, that won’t take too long, I have some plans for that as I mentioned in the last email :)

[ghost]

Alright, I have added the change in the development branch. Thanks!