revanced-website icon indicating copy to clipboard operation
revanced-website copied to clipboard

fix: add footer to home

Open oSumAtrIX opened this issue 2 years ago • 9 comments

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.

oSumAtrIX avatar Aug 21 '23 21:08 oSumAtrIX

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:

  • two socials (twitter and youtube - I don't think we really need to list them)

  • an email (again, I don't think this needs to be on the home page when you can find it on the download page)

  • this blurb of text:

    ReVanced was born out of Vanced's discontinuation and it is our goal to continue the legacy of what Vanced left behind. Thanks to ReVanced Patcher, it's possible to create long-lasting patches for nearly any Android app. ReVanced's patching system is designed to allow patches to work on new versions of the apps automatically with bare minimum maintenance.

    which I think could be integrated into the home page's copy, given the current copy isn't very descriptive.

KTibow avatar Jan 16 '24 14:01 KTibow

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 avatar Jan 17 '24 21:01 oSumAtrIX

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

oSumAtrIX avatar Jan 17 '24 21:01 oSumAtrIX

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.

KTibow avatar Jan 17 '24 22:01 KTibow

@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

oSumAtrIX avatar Jan 17 '24 22:01 oSumAtrIX

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.

KTibow avatar Jan 17 '24 22:01 KTibow

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?

oSumAtrIX avatar Jan 17 '24 22:01 oSumAtrIX

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.

KTibow avatar Jan 17 '24 22:01 KTibow

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

oSumAtrIX avatar Jan 17 '24 22:01 oSumAtrIX