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

Issue - heatmap is mini-heatmap #13634

Closed
4 tasks done
wULLSnpAXbWZGYDYyhWTKKspEQoaYxXyhoisqHf opened this issue Nov 19, 2020 · 7 comments · Fixed by #13645
Closed
4 tasks done

Issue - heatmap is mini-heatmap #13634

wULLSnpAXbWZGYDYyhWTKKspEQoaYxXyhoisqHf opened this issue Nov 19, 2020 · 7 comments · Fixed by #13645
Labels
issue/confirmed Issue has been reviewed and confirmed to be present or accepted to be implemented type/bug

Comments

@wULLSnpAXbWZGYDYyhWTKKspEQoaYxXyhoisqHf
Copy link
Contributor

  • Gitea version (or commit ref): Gitea version 1.14.0+dev-239-gad2a28862 built with GNU Make 4.3, go1.15.5 : bindata, sqlite, sqlite_unlock_notify
  • Git version: git version 2.29.2.497.ga2da47d1e7
  • Operating system: Arch Linux
  • Built from:
    • source
  • Run with:
    • systemd
  • Database:
    • SQLite
  • Can you reproduce the bug at https://try.gitea.io:
    • Yes

Description

Appears to be introduced by #13623 .
The issue is that the heatmap appears really small.
I have tried to reproduce it in chromium, firefox and brave (chromium-based) and this issue only pertains in chromium (consistently).
For the user dashboard to pop up you have to be logged in, in chromium, though, you can also see the issue when navigating to user's 'Public Activity' page (https://try.gitea.io/kjhfdskahhjksdf?tab=activity), for which you, obviously, don't have to be logged in.

Screenshots

my instance
image
try.gitea.io logged in (Public Activity) pic1
image
try.gitea.io logged in (dashboard) pic2
image
try.gitea.io logged out (activity)
image

edit: missing parenthesis (image not showing properly)
edit2: add blame

@6543 6543 added type/bug type/refactoring Existing code has been cleaned up. There should be no new functionality. issue/confirmed Issue has been reviewed and confirmed to be present or accepted to be implemented and removed type/refactoring Existing code has been cleaned up. There should be no new functionality. labels Nov 19, 2020
@6543
Copy link
Member

6543 commented Nov 19, 2020

@silverwind some sort of chrome css incompatiblety?

@wULLSnpAXbWZGYDYyhWTKKspEQoaYxXyhoisqHf
Copy link
Contributor Author

@wULLSnpAXbWZGYDYyhWTKKspEQoaYxXyhoisqHf I dont have this issue :/ it looks like you have to clear your browser cache

I already did...

@6543
Copy link
Member

6543 commented Nov 19, 2020

@wULLSnpAXbWZGYDYyhWTKKspEQoaYxXyhoisqHf sorry - didnt read the " ... chrome ..." the first time ...

@wULLSnpAXbWZGYDYyhWTKKspEQoaYxXyhoisqHf
Copy link
Contributor Author

@wULLSnpAXbWZGYDYyhWTKKspEQoaYxXyhoisqHf sorry - didnt read the " ... chrome ..." the first time ...

no worries.
and I only tried chromium.
don't use chrome so can't confirm it's present there (since the issue is e.g. not present in chromium-based brave).

@silverwind
Copy link
Member

I think I only tested this in Firefox if I recall correctly.

silverwind added a commit to silverwind/gitea that referenced this issue Nov 19, 2020
Apparently SVG inside flexbox renders slightly different across browsers
where Firefox would stretch to fit the parent while Chrome and safari
wouldn't. Stretch the SVG to the width of the parent for consistent
rendering.

Also did a few minor tweaks on the min-height of the box so it takes up
less space on smaller responsive breakpoints.

Fixes: go-gitea#13634
Fixes: go-gitea#13637
wULLSnpAXbWZGYDYyhWTKKspEQoaYxXyhoisqHf pushed a commit to wULLSnpAXbWZGYDYyhWTKKspEQoaYxXyhoisqHf/gitea that referenced this issue Nov 19, 2020
Apparently SVG inside flexbox renders slightly different across browsers
where Firefox would stretch to fit the parent while Chrome and safari
wouldn't. Stretch the SVG to the width of the parent for consistent
rendering.

Also did a few minor tweaks on the min-height of the box so it takes up
less space on smaller responsive breakpoints.

Fixes: go-gitea#13634
Fixes: go-gitea#13637
@bkraul
Copy link

bkraul commented Nov 23, 2020

Any chance #13645 be merged soon? I'd be happy with the current heatmap if I was an ant right now, but since I am not... 🙂

@wULLSnpAXbWZGYDYyhWTKKspEQoaYxXyhoisqHf
Copy link
Contributor Author

Any chance #13645 be merged soon? I'd be happy with the current heatmap if I was an ant right now, but since I am not...

for now, you could build Gitea with the patch (that's what I've done)...

cd gitea-src-folder
curl -O https://patch-diff.githubusercontent.com/raw/go-gitea/gitea/pull/13645.patch
git am 13645
make build # specify desired TAGS before this...

@bkraul

edit silly typo: git am 12645 --> git am 13645

techknowlogick pushed a commit that referenced this issue Nov 23, 2020
* Fix heatmap rendering in Chrome and Safari

Apparently SVG inside flexbox renders slightly different across browsers
where Firefox would stretch to fit the parent while Chrome and safari
wouldn't. Stretch the SVG to the width of the parent for consistent
rendering.

Also did a few minor tweaks on the min-height of the box so it takes up
less space on smaller responsive breakpoints.

Fixes: #13634
Fixes: #13637

* position tweak

Co-authored-by: zeripath <art27@cantab.net>
@go-gitea go-gitea locked and limited conversation to collaborators Jan 18, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
issue/confirmed Issue has been reviewed and confirmed to be present or accepted to be implemented type/bug
Projects
None yet
4 participants