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

SD-1157 Web: Rozdelit darcov po rokoch #22

Open
wants to merge 4 commits into
base: master
Choose a base branch
from

Conversation

michal-rohacek
Copy link
Contributor

No description provided.

Copy link
Contributor

@eronisko eronisko left a comment

Choose a reason for hiding this comment

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

Kod fajn, len vizualne by som to este trocha pomenil.

public/js/main.js Outdated Show resolved Hide resolved
<div class="col-sm-3"><a href="/podporovatelia/2019">Podporovatelia v roku 2019</a></div>
<div class="col-sm-3"><a href="/podporovatelia/2018">Podporovatelia v roku 2018</a></div>
<div class="col-sm-3"><a href="/podporovatelia/2017">Podporovatelia v roku 2017</a></div>
<div class="col-sm-3"><a href="/podporovatelia/2016">Podporovatelia v roku 2016</a></div>
Copy link
Contributor

Choose a reason for hiding this comment

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

Toto nie je uplne stastne riesenie. Tie prepinatka vyzeraju nefunkcne, lebo loga sa nemenia, len zoznamy pod nimi. Pridal by som ten rok aspon do nadpisu, nech je vidno zmenu.

Pod napis by som potom mozno hodil tie linky dakto takto:
image

A zoznam firiem je up-to-date, alebo zatial cakas na data?

Copy link
Contributor

Choose a reason for hiding this comment

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

Este by som navrhol namiesto /podporovatelia/2019 spravit /podporovatelia (a dotiahnut tam podporovatelov za r. 2019). Podstranky tym padom budu len na minule roky. cc @jsuchal

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Cakam na data, zoznam firiem je vsade rovnaky len ako placeholder.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Malo by to byt. Az na tie firmy

Copy link
Member

Choose a reason for hiding this comment

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

@michalrohacek256 skusis nieco vytiahnut z vyrocnych sprav? https://slovensko.digital/vyrocne-spravy

Copy link
Contributor Author

Choose a reason for hiding this comment

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

pozriem sa na to

Copy link
Contributor Author

Choose a reason for hiding this comment

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

presiel som tie vyrocne spravy - su v nich len loga firiem (ako na stranke, grid bez uvedenia prispevkov) a nizsie nejake sumare, ale menovite firmy s jednotlivymi sumami som nenasiel

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.

3 participants