Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix: add footer to home #175

Open
oSumAtrIX opened this issue Aug 21, 2023 · 9 comments
Open

fix: add footer to home #175

oSumAtrIX opened this issue Aug 21, 2023 · 9 comments
Assignees
Labels
Bug report Something isn't working

Comments

@oSumAtrIX
Copy link
Member

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 oSumAtrIX added the Bug report Something isn't working label Aug 21, 2023
@oSumAtrIX oSumAtrIX assigned KTibow and unassigned xafn Jan 7, 2024
@KTibow
Copy link
Contributor

KTibow commented Jan 16, 2024

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.

@oSumAtrIX
Copy link
Member Author

oSumAtrIX commented Jan 17, 2024

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
Copy link
Member Author

oSumAtrIX commented Jan 17, 2024

PR #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
Copy link
Contributor

KTibow commented Jan 17, 2024

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
Copy link
Member Author

@KTibow I am not sure if I understood you correctly. Here is a new image link #214 (review)

@KTibow
Copy link
Contributor

KTibow commented Jan 17, 2024

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
Copy link
Member Author

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
Copy link
Contributor

KTibow commented Jan 17, 2024

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
Copy link
Member Author

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 oSumAtrIX assigned Ushie and unassigned KTibow Jun 3, 2024
@oSumAtrIX oSumAtrIX assigned xafn and unassigned Ushie Jun 29, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug report Something isn't working
Projects
Status: Backlog
Development

Successfully merging a pull request may close this issue.

4 participants