Exifly / ApiVault

Your gateway to a world of public APIs.

https://apivault.dev

Other

444 stars 40 forks source link

fix-Hidden TRENDING element in mobile view #123

Closed SamantaTarun closed 1 year ago

SamantaTarun commented 1 year ago

The title "TRENDING" is visible on the mobile version, and it should be hidden.

To Reproduce

Open the website on a mobile device. Navigate to the homepage. Observe that the title "TRENDING" is visible.

113

I have added it as per iPhone 12 screen max-width. If any changes required let me know.

SamantaTarun commented 1 year ago

@gdjohn4s could you please let me know what is the expected max width for your device as you have mentioned device as iphone12 so I did according to the iPhone 12 max-width.

gdjohn4s commented 1 year ago

The resolution displayed at the top of the screen I sent you in the previous comment should be fine for all devices with a smaller resolution than most phones.

375x667

FlavioAdamo commented 1 year ago

We are planning to release a new version of the software today, so to not lose your work, we will approve your PR and fix the issuses. Thanks for the contribute <3

SamantaTarun commented 1 year ago

@gdjohn4s please add me as a member of your organization. I wanna contribute more in tge future.