sacredboard icon indicating copy to clipboard operation
sacredboard copied to clipboard

Result lists should be sorted by first entry

Open Qwlouse opened this issue 8 years ago • 4 comments

ATM if sorting by results that are lists sorts by the last element of that list. Should be the first instead.

Qwlouse avatar May 18 '17 12:05 Qwlouse

I'm unable to replicate the issue. Can you step through what you did?

femmebot avatar Aug 13 '15 18:08 femmebot

Narrow a window from the left side and as the content begins collapsing and responding normally, watch for a horizontal scroll bar to pop up. It may even appear and then disappear at a smaller size if you keep going, it seems dependent on the random actions of the script for The Fox & The Grapes. If it the horizontal scroll doesn't appear, you can usually prompt it by moving the grapes around with the fox cursor - it'll break out of its parent to the right. I'm in the latest version of Chrome.

screen shot 2015-08-13 at 4 13 55 pm The scroll appears

screen shot 2015-08-13 at 4 14 33 pm The scroll lengthens as the grapes are pushed out to the right when the window is made even narrower

Excuse my messy WIP in the back haha.

laurelobrien avatar Aug 13 '15 23:08 laurelobrien

Hmm, I have been resizing the width but still haven't been able to replicate. I'll keep digging around. Curious, what is the little watermark/icon on the lower bottom right corner of your browser? It shows up on both screenshots you took where the cursor escapes the boundary of the page canvas.

(BTW the WIP in the back looks fab!)

femmebot avatar Aug 14 '15 17:08 femmebot

I'll keep poking around myself: blocking out the script and relevant html in a local version solves it for me, but neither one by themselves. I may have been wrong about the culprit the more that I look at it, but it seems like the only element on the page whose width was not specifically the current width of the page or smaller (it's auto according to Chrome dev tools, but I'm not sure how the width of js scripts work).

It's a Chrome extension for Feedly, an rss feed, but it does still happen with extensions disabled.

Thanks!

laurelobrien avatar Aug 14 '15 17:08 laurelobrien

Aha! I was wrong. Another element was breaking the width at smaller sizes, which simply allowed the grape to bounce outside its boundaries and increase the horizontal scroll already present. It looks like the .container for .section-fox-lion is set to width: 960px; and sticks to that when everything else has scaled down. It's inheriting the value from the general styles at the top of the stylesheet - changing .container to max-width: 960px; fixes it for me. Scrollbar disappears and everything is collapsed quite elegantly. Interesting because several other compositions inherit the same .container but don't have that problem. Perhaps they have more stringent media queries.

screen shot 2015-08-14 at 10 53 36 am width: 960px; persists

screen shot 2015-08-14 at 11 25 25 am changing it to max-width: in dev tools removes the scroll and everything looks as it should in narrower browsers

Not sure if this is a sustainable solution as the general .container was probably styled that way for a reason, and other compositions have no problems with it. Also strange you can't replicate it, hmm.

laurelobrien avatar Aug 14 '15 18:08 laurelobrien

I too can see the horizontal scroll appear in Chrome, it is present from 660px through 980px browser width. I can't replicate the grapes shifting to the right though.

dschiffner avatar Aug 14 '15 20:08 dschiffner

I just pushed an update—changing the .container width to max-width: 960px—to, hopefully, address the horizontal scroll. I'm still unable to get the grapes to move off the page so I can't tell if that resolved it.

femmebot avatar Aug 15 '15 05:08 femmebot

Hey all—hopping in here as I created the original section.

Great catch @laurelobrien - I was able to replicate using the steps you outlined.

@femmebot: I've added an additional fix here: https://github.com/femmebot/google-type/pull/39 which should cover any browser resizing/document flow breaking from the grapes.

@laurelobrien, @dschiffner: once the pull RQ is in, please verify everything is working for you.

As a side note, most users won't be using the responsive site like this: http://i.imgur.com/YkbaV.gif :wink:

twahlin avatar Aug 15 '15 13:08 twahlin

@laurelobrien (also, the WIP is looking good)

twahlin avatar Aug 15 '15 13:08 twahlin

I just merged fix #39 Thanks @twahlin!

femmebot avatar Aug 15 '15 17:08 femmebot

Aha, no worries, that's just my quick n dirty way of seeing when and if things break as they size down before seeing if the actual device gets the problem too. Sometimes it catches the browser in awkward breakpoints that don't "exist" though, so to speak. It's working for me, looks great!

laurelobrien avatar Aug 15 '15 17:08 laurelobrien

Thanks all!

twahlin avatar Aug 16 '15 17:08 twahlin