feat: add legal pages and copyright notice #543
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
RicoPleasure
changed the title
RicoPleasure
requested review from
joaodiaslobo,
nunom27,
FilipeR13 and
AfonsoMartins26
1 task
FilipeR13
requested changes
Nov 25, 2024
FilipeR13
requested changes
Nov 25, 2024
FilipeR13
requested changes
Nov 25, 2024
FilipeR13
requested changes
Dec 23, 2024
| </div> | ||
| <!-- Link to Home (hidden on mobile) --> | ||
| <div class="flex place-items-center lg:pr-12"> | ||
| <.link class="atomic-button atomic-button--white atomic-button--md hidden sm:block" navigate={~p"/"}>Back home</.link> |
There was a problem hiding this comment.
Change this link to the button component. Make use of the attribute patch so you mantain the funcitionality of the link.
Put the text Back Home in a gettext("Back Home")
| <!-- Link to Home (hidden on mobile) --> | ||
| <div class="flex place-items-center lg:pr-12"> | ||
| <.link class="atomic-button atomic-button--white atomic-button--md hidden sm:block" navigate={~p"/"}>Back home</.link> | ||
| <.link class="atomic-button atomic-button--md hero-home block text-zinc-400 sm:hidden" navigate={~p"/"}></.link> |
There was a problem hiding this comment.
not sure if the component button allows only a icon without text, if so, change this too.
| ~H""" | ||
| <section class="flex justify-center bg-zinc-800 p-10 font-semibold text-white"> | ||
| <p class="flex w-full justify-center md:w-2/3"> | ||
| Lorem ipsum dolor sit amet. |
There was a problem hiding this comment.
Suggested change
| Lorem ipsum dolor sit amet. | |
| <%= getttext("Lorem ipsum dolor sit amet.") %> |
Comment on lines
+11
to
+18
| <p class="text-xs font-semibold text-zinc-400 hover:underline">Terms of Service</p> | ||
| </.link> | ||
| <.link navigate={~p"/privacy"} class="shrink-0 select-none"> | ||
| <p class="text-xs font-semibold text-zinc-400 hover:underline">Privacy Policy</p> | ||
| </.link> | ||
| <.link navigate={~p"/cookies"} class="shrink-0 select-none"> | ||
| <p class="text-xs font-semibold text-zinc-400 hover:underline">Cookie Policy</p> | ||
| </.link> |
There was a problem hiding this comment.
Add gettexts to the text on the links
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Description
Created pages for Terms of Service, Privacy Policy and Cookie Policy. Added links to them below the schedule when on large screens and at the bottom of the sidebar menu when on mobile.
Pages
Links