Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

date-range Facet limit histogram is weird #270

Closed
eddierubeiz opened this issue Aug 1, 2019 · 12 comments
Closed

date-range Facet limit histogram is weird #270

eddierubeiz opened this issue Aug 1, 2019 · 12 comments
Assignees
Labels
bug Something isn't working

Comments

@eddierubeiz
Copy link
Contributor

  1. Start here
  2. Click Date in the faceting apparatus on the left
  3. Type 1973 into the lower bound for dates
  4. Click the apply button to limit dates.
  5. The HTML is broken. Looks like malformed HTML.
@jrochkind
Copy link
Contributor

oh man I ran into something similar that eventually went away, and I was never able to fix it.

Something is weird with the code that is using JS canvas to plot that graph, and deciding how wide the graph should be.

I am not sure why it doesn't occur on chf_sufia, but Blacklight's own CSS changed. It is hard to reproduce. I'm not sure if it is reproducible on stock latest Blacklight and blacklight_range_limit.

This may be a hard one to fix. blacklight_range_limit is some hacky stuff.

@jrochkind
Copy link
Contributor

hmm, 1973 isn't a magic number though, seems to be happening with nearly anything. This is indeed bad. gah.

@eddierubeiz
Copy link
Contributor Author

Yeah, this is generalized. I came across it working on my featured topics page.

@eddierubeiz eddierubeiz added the bug Something isn't working label Sep 18, 2019
@eddierubeiz eddierubeiz changed the title Bug: facet limit form Facet limit form Sep 18, 2019
@jrochkind jrochkind changed the title Facet limit form date-range Facet limit histogram is weird Oct 2, 2019
@jrochkind
Copy link
Contributor

Note could not be reproduced on Windows Chrome on 5th floor conference room.

i also noticed that clicking on the "Date" heading to collapse and then again to expand the date limit area -- gets it to be sized correctly again.

@HKativa
Copy link

HKativa commented Oct 30, 2019

Note: I did reproduce this bug on Windows Chrome on my office computer while testing Kithe on 10/30/2019.

Date_histogram_error.jpg

@jrochkind
Copy link
Contributor

Filed with blacklight_range_limit at projectblacklight/blacklight_range_limit#111

@HKativa
Copy link

HKativa commented Oct 31, 2019

Note in case this isn't a known issue: this error occurs when using the "limit dates" feature on the homepage as well.

Date_homepage_search.jpg

Select "go" and you get:

Date_histogram_error_homepage.jpg

@jrochkind
Copy link
Contributor

Yep, it's going to happen on any page that is displaying a pre-chosen date limit. At least it can potentially happen on any such page; we know that it is a somewhat intermittent/unpredictable problem.

jrochkind added a commit that referenced this issue Nov 4, 2019
It is one way to fix #270. Does it make the page load much different? We will investigate with chrome performance tools and see.
@jrochkind
Copy link
Contributor

I'm currently not able to reproduce it, it no longer happens for me. Which makes it hard to tell if a change I've made fixes it or not, when I can no longer make it happen without the change.

@eddierubeiz @HKativa are you currently seeing it on your own machines?

@eddierubeiz
Copy link
Contributor Author

eddierubeiz commented Nov 5, 2019

I CANNOT reproduce it! It works for me in Safari and Chrome. As far as my browser is concerned... either you fixed it or it fixed itself.

@sanfordd
Copy link

sanfordd commented Nov 5, 2019

Cannot reproduce on Chromium or Firefox looks fine for me.

@jrochkind
Copy link
Contributor

It's just hard to reproduce, becuase it's basically a "race condition" that occurs depending on unpredictable timing of browser/network events. That we can't see it now doesn't mean it won't come back, but it does make it hard to work on, since it's hard to be sure if a code change fixed it when you can't reproduce it.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

4 participants