@hlfan commented on this pull request.
> -
- cal.on("mouseover", (event, timestamp, value) => {
- if (!displayName || !value) return;
- if (event.target.parentElement.nodeName === "a") return;
+ }
+ if (weekColumn === 1) {
+ $day.addClass("d-none");
+ continue;
+ }
+ const count = $day.data("count") ?? 0;
+ const tooltipText = getTooltipText(date, count);
+ $day
+ .css("grid-area", getWeekdayRow(date.getUTCDay()) + " / " + weekColumn)
+ .attr("aria-label", tooltipText)
+ .tooltip({
+ title: tooltipText,
We don't know how wide the tooltip will get, so why 300 px instead of `none`?
--
Reply to this email directly or view it on GitHub:
https://github.com/openstreetmap/openstreetmap-website/pull/5998#discussion_r2089593820
You are receiving this because you are subscribed to this thread.
Message ID:
<openstreetmap/openstreetmap-website/pull/5998/review/2841270...@github.com>_______________________________________________
rails-dev mailing list
rails-dev@openstreetmap.org
https://lists.openstreetmap.org/listinfo/rails-dev
