ReVanced / revanced-website

🌐 Website for ReVanced
https://revanced.app
GNU General Public License v3.0
313 stars 42 forks source link

fix: add footer to home #175

Open oSumAtrIX opened 1 year ago

oSumAtrIX commented 1 year ago

Issue

Currently, every page except the home page has a footer. This is unconventional and may also impact SEO.

Solution

Add the same footer to the home page as the other pages too.

Additional context

It may be the case that the footer has not been added to the home page for some design-specific reasons. In this case, a solution has to be found, as it is unconventional not to include the footer and impacts SEO.

KTibow commented 10 months ago

I don't believe this is a problem for SEO or for users. Most content in the footer can be found on the home page already. The only things missing are:

oSumAtrIX commented 10 months ago

Google, for example, reads the footer section for certain attributes to display the website, which already justifies a footer on the index page from the perspective of SEO. Apart from that, adding the footer restores consistency with other pages (why have an if (index) hideFooter() logic at all to begin with?) and ensures every page contains important information which you identified correctly in your previous comment (Socials, links, especially the email, and the about). Lastly, having a footer on the index page is quite the standard practice that users find on every page, including the index page.

From my POV, I lack concrete motivation from your side that speaks against these arguments.

oSumAtrIX commented 10 months ago

PR https://github.com/ReVanced/revanced-website/pull/214 has been closed as it failed regarding the design (Particularly, the wave could have blended better with the footer). For that reason, when we decide to go forward with this issue, we will need to figure out a solution in terms of design. An initial proposal from my end is to make the wave appear at the bottom, just like it does at the top. This would prevent the "cutoff" that is visible here:

image

KTibow commented 10 months ago

I don't think this will be easy to integrate without messing with the footer or homepage, given that we currently have the screenshot being cut off at the bottom. Either we unnaturally don't show the rest of the screenshot, change how the screenshot is shown, or make the footer only use the space to the left of the screenshot. The "image" link you sent must have expired, because it doesn't work.

oSumAtrIX commented 10 months ago

@KTibow I am not sure if I understood you correctly. Here is a new image link https://github.com/ReVanced/revanced-website/pull/214#pullrequestreview-1804022216

KTibow commented 10 months ago

This is what the home page looks like on my laptop: image You can see that if there was a theoretical footer, the footer would intersect with it.

oSumAtrIX commented 10 months ago

Yes. As part of my last comment that referred to this issue, I have added an initial design idea that could potentially solve it or act as a basis for better ideas and solutions that are yet to be found, or used until a solution is present. Do I understand that you have a question in regards to that?

KTibow commented 10 months ago

Even if there was a wave, the screenshot would still go on top of the wave and overlap the content. The main solution I see is to make the screenshot go under the wave.

oSumAtrIX commented 10 months ago

The bottom wave as I imagine it would take more height than the image so that the image does not overlap with the footer, alternative a scroll event can percentually shift the image up