langx / website

This is the official website of langx.io
https://langx.io
MIT License
16 stars 9 forks source link

🐛 [BUG] : Flowing github star counter in header #31

Open xuelink opened 7 months ago

xuelink commented 7 months ago

Describe the bug Flowing github star counter

Expected behavior It should stick to theme toggle

Screenshots

Simulator Screenshot

Smartphone (please complete the following information): iPhone SE

Additional Related with #20

If you want speed up process of this issue, please send 🚀 as a reaction.

neebooo commented 7 months ago

Can you assign this to me please

xuelink commented 7 months ago

Can you assign this to me please

Done! Thank you for interest.

xuelink commented 5 months ago

Can you assign this to me please

We miss you @neebooo :) Please, tell me if you can't have time to to that, let me do this issue free to be assignable.

xuelink commented 5 months ago

I think you can fix this bug @jakebogan01 , let me know if you want to be assigned this issue as well Best

jakebogan01 commented 5 months ago

@xuelink Yeah I can try this one. Let me finish the footer issue first and then I can attempt this issue next

jakebogan01 commented 5 months ago

@xuelink I have viewed the website on all major browsers, mobile and desktop. I can't replicate this bug, nor do I even see the GitHub star in general. Is this still an issue somehow?

xuelink commented 5 months ago

@jakebogan01

https://github.com/langx/website/assets/3280475/84b3bea1-d32d-4095-9fd2-c8f995c8d670

xuelink commented 5 months ago

You may play with CSS to make github icon and number horizontally instead of vertically, it may fix 😉

jakebogan01 commented 5 months ago

@xuelink Thank you for the video, I was testing on the live site. I didn't know it was commented out.

I made a PR for this issue here

xuelink commented 5 months ago

@jakebogan01 did you check with big numbers :D something like more than 1000+, is it gonna be 1K :D we will be soon hope :)

jakebogan01 commented 5 months ago

@xuelink Yes, I made sure that it can contain as many numbers as needed. I also made sure that a comma will be in place for example: 45,000

Maybe down the line we can write some logic that replaces 4,000 with 4k and so on. But for now it'll show 4,000

xuelink commented 5 months ago
Screenshot 2024-04-18 at 10 19 50 PM
https://api.github.com/repos/appwrite/appwrite?page=$i&per_page=100

its ok, np :) thank you! @jakebogan01 hope we will see these days as well :)

jakebogan01 commented 5 months ago

No problem, hope so haha

xuelink commented 5 months ago

I realized with links like cookies or other pages, github star counter doesnt work, but with /blog it works @jakebogan01

https://github.com/langx/website/assets/3280475/5ef39828-2ddd-4ea4-a277-60d0be9aca39

jakebogan01 commented 5 months ago

@xuelink that's weird. I never moved any of the api fetch logic, I only added the github icon. I did move the github count from outside the icon to inside of it but that shouldn't have broken it. I'll take a look and see what I can find out about that

jakebogan01 commented 5 months ago

@xuelink Locally, the github stars works on all pages, blogs, privacy, terms and cookies and so on. But on the live website, cookies is the only page that its not working for me. Not sure why, I don't even see a status code in the network tab

xuelink commented 5 months ago

@xuelink Locally, the github stars works on all pages, blogs, privacy, terms and cookies and so on. But on the live website, cookies is the only page that its not working for me. Not sure why, I don't even see a status code in the network tab

@DH-555 What do you think about that ? Maybe its related with Cloudflare :/ i have no idea

DH-555 commented 5 months ago

No idea, will check by the weekend, but it seems that it's not being made any request in that specific page in order to fetch the data from GitHub.