Projekt

Obecné

Profil

Podání #23304

uzavřený

Github Issue: (praha.pirati.cz) Thumb profilové fotografie

Přidáno uživatelem Jan Loužek před asi 5 roky(ů). Aktualizováno před asi 5 roky(ů).

Stav:
Dokončen
Priorita:
Normální
Přiřazeno:
Kategorie:
-
Začátek:
30.09.2019
Uzavřít do:
% Hotovo:

0%

Odhadovaná doba:

Popis

Ahoj,

pro stránku lidé https://praha.pirati.cz/lide/ máme agregovaná data z jednotlivých vyplněných boxů. To je velmi chytré řešení ale vytváří jeden problém: Všichni, kteří mají být uvedeni na jedné stránce musí mít vyplněny hodnoty stejným způsobem, jinak to vypadá špatně (jak dokládá tento commit: https://github.com/pirati-web/praha.pirati.cz/commit/8591ec92dc078da1466556b790540799ef15cacc

Rád bych ale vyřešil fotky. Bylo by možné použít pro tuto stránku jinou sadu fotek? Tedy, aby byl thumb jiný než fotka z profilu? Mohlo by se to vyřešit poměrně jednoduše tím, že by se do boxu zavedl nový parametr thumbimg: (stejně jako již máme img:) Jde mi o to, že naši zastupitelé chtějí mít svoje oblíbené fotografie na svých profilech, ale zároveň chtějí na přehledové stránce Lidé https://praha.pirati.cz/lide/ aby byly fotografie s jednotným pozadím, které jsou připravené, ale nejsou zatím pouze nasazené.

Díky za vyřešení tohoto požadavku.


Soubory

Aktualizováno uživatelem Jan Loužek před asi 5 roky(ů)

  • Předmět změněn z Redmine Issue: (praha.pirati.cz) Thumb profilové fotografie na Github Issue: (praha.pirati.cz) Thumb profilové fotografie

Aktualizováno uživatelem Andrej Ramašeuski před asi 5 roky(ů)

nepomohl by vam externi storage fotek s resizem?

Aktualizováno uživatelem Jan Loužek před asi 5 roky(ů)

Ahoj, toho se to aktuálně netýká :-) Ty fotografie mají každá pod 1 MB, já tu jen žádám o navržení toho parametru. Důvod je ten, že zastupitelé chtějí jiné fotografie na svém profilu než jaké je potřeba, aby byly na podstránce Lidé. Proto potřebuju, aby se to dalo rozlišit parametrem. Dík!

Aktualizováno uživatelem Andrej Ramašeuski před asi 5 roky(ů)

z dlohodobe perspektivy, ale, uvazujeme o presub assetu, predevsim fotografii ven z git repozitaru. tam by prezili nejen napric ruznymi systemy, ale i mezi ruznymi verzema webu.

Aktualizováno uživatelem Jan Loužek před asi 5 roky(ů)

To jo. Rozumím, že je potřebné toto vyřešit, pak repozitáře bobtnají, jsou pomalejší, zažil jsem to. Jinak můj podnět k issue se ale tohoto netýkal :-) Abychom tu diskuzi nestočili tam kam by připadnout neměla.

Aktualizováno uživatelem Jan Loužek před asi 5 roky(ů)

Ahoj. Chtěl bych jen uvedenou věc připomenout, zastupitelé se mně ptají, jak se daná věc realizuje. Mohl by být prosím parametr zaveden? Děkuji.

Aktualizováno uživatelem Jan Loužek před asi 5 roky(ů)

  • Přiřazeno nastaven na Jitka Novotná

Aktualizováno uživatelem Jitka Novotná před asi 5 roky(ů)

Ten kód, který říká, kde se berou fotky je v thema. Konkrétně tady:

https://github.com/pirati-web/jekyll-theme-pirati/blob/master/_includes/people/profile-badge.html#L12

když to chceš změnit v konkrétním webu tak je třeba ten soubor zkopírovat (což už se na Praze stalo) a změnit tam.

Tj přepsat žádky 14 a 16 případne je obalit 'if'. Zvládneš to sám, nebo potřebujěš pomoct?

Ještě to má háček, ten profile-badge se používá i jinde, třeba v kontaktech. Chceš ty fotky měnit ve všech profile-badge nebo jen v members? (snažší je to měnit všude)

Aktualizováno uživatelem Jan Loužek před asi 5 roky(ů)

Asi měnit všude. Rozumím tomu, že tedy když přidám nové fotky do /assets/images/people a pod img: dám ještě profile-badge: tak budou v těch thumbech jiné? Upřímně řečeno jsem trochu tomuto: Tj přepsat žádky 14 a 16 případne je obalit 'if'. Zvládneš to sám, nebo potřebujěš pomoct neporozumněl :-(

Díky

Aktualizováno uživatelem Jitka Novotná před asi 5 roky(ů)

Tak jsem to tam přidala, teď by to mělo fugovat jak jsi navrhoval. Tj když vyplníš profile-badge tak ji vezme namísto img. Ještě jsem to neotestavala, tak buť u prvního opatrný, ale mělo by to fungovat.

Aktualizováno uživatelem Jan Loužek před asi 5 roky(ů)

Děkuju moc, boží! Jdu vyzkoušet u některého uvolněného zastupitele ;-)

Aktualizováno uživatelem Jitka Novotná před asi 5 roky(ů)

omlouvám, se už pushuju fix, měla jsem to otestovat

Aktualizováno uživatelem Jan Loužek před asi 5 roky(ů)

Díky, už to funguje, můžeme tento úkol uzavřít.

Aktualizováno uživatelem Jan Loužek před asi 5 roky(ů)

Již vyřešeno, možné ukončit. Děkuju!

Aktualizováno uživatelem Jitka Novotná před asi 5 roky(ů)

  • Stav změněn z Nový na Dokončen

Také k dispozici: Atom PDF