Skip to content

Update footer: show only single address #425

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

Merged
merged 4 commits into from
Aug 5, 2024

Conversation

jsmitka
Copy link
Contributor

@jsmitka jsmitka commented Aug 5, 2024

Při první úpravě jsem se zaměřil hlavně na tu fakturační adresu a tu druhou nezměnil. Patičku jsem tedy upravil, aby tam byla jen jedna adresa. Zároveň jsem jí o trošku přeskládal, aby to vypadalo lépe, udělal jsem ikonky u souborů ke stažení klikatelné a vyladil bílé místo na mobilu.

Nová podoba patičky:

Pyvec neziskovka české Python komunity - Vivaldi-2024-08-05-13-02-16

jsmitka added 4 commits August 5, 2024 12:46
Previously, only the billing address was updated.
Footer changes:

* Show only one address.
* Show download links as a list, make their icon clickable.
* Move download links to their own column.

The change included small tweaks to CSS, so a version query string
was added to prevent stale caches.
Copy link
Member

@honzajavorek honzajavorek left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Krása, děkujeme!

Ječná 507/6<br>
12000 Prague<br>
Czech Republic, EU
directions: *address
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hustý, YAML MacGyver!

@@ -10,35 +10,28 @@ claim:
Service non-profit organization of&nbsp;the&nbsp;Czech<br>
[Python](https://python.cz/en/)&nbsp;programming&nbsp;language user&nbsp;group

# Billing address is not used in the template, but it was part of the JSON API response.
# Keeping it does not hurt us, therefore we should keep it for compatibility.
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Už jsem úplně zapomněl, že tam je „API“ 😅 Nejsem si teda jistý, zda se k něčemu někde používá...

@honzajavorek honzajavorek merged commit 92c804a into pyvec:master Aug 5, 2024
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants