search

DevLeonardoCommunity / github-stats

Aggregating and displaying YOUR GitHub Stats in meaningful metrics (we learn web development in the meantime)
https://public-github-stats.vercel.app
MIT License
71 stars 37 forks source link

fix: stats layout and add tooltips #101

Closed theflucs closed 9 months ago

theflucs commented 9 months ago

Fix responsive card layout Add tooltips to pr and contribution title (removed title tag), so fixing the issue #95

netlify[bot] commented 9 months ago

Deploy Preview for public-github-stats ready!

Name Link
Latest commit 08e2254c2d11f2fd5c479277b85fb924e5772fef
Latest deploy log https://app.netlify.com/sites/public-github-stats/deploys/65b8bec83745a900080d4e08
Deploy Preview https://deploy-preview-101--public-github-stats.netlify.app
Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

theflucs commented 9 months ago

@CBID2 are you sure your code is aligned with this branch? The PR title shows indeed the 3 dots, if it is too long, and also the contribution icons on the right should be aligned with the PR title pill, they don't look like. The tooltip part I think is fixed for the PR title but not for the contribution title, as mentioned above by @Balastrong. You can have a look at it and see if you come up with a solution. Thanks!

CBID2 commented 9 months ago

@CBID2 are you sure your code is aligned with this branch? The PR title shows the 3 dots, if too long, and also the contribution icons on the right should be aligned with the PR title pill, they don't look like. The tooltip part I think is fixed for the PR title but not for the contribution title, as mentioned above by @Balastrong. You can have a look at it and see if you come up with a solution

@Balastrong this deployment needs approval again.

Balastrong commented 9 months ago

@CBID2 the deploy is not working at the moment, it redirects to an old version.

In order to test this branch you should checkout it locally and run the app there :)

Balastrong commented 9 months ago

@all-contributors please add @CBID2 for review

allcontributors[bot] commented 9 months ago

@Balastrong

I've put up a pull request to add @CBID2! :tada:

CBID2 commented 9 months ago

@all-contributors please add @CBID2 for review

@Balastrong, you used my name instead of @theflucs. Did you mean to do that?

allcontributors[bot] commented 9 months ago

@CBID2

@CBID2 already contributed before to review

Balastrong commented 9 months ago

Yes, I added you for the review :D