Skip to content

Conversation

@gardenerik
Copy link
Contributor

@gardenerik gardenerik commented Apr 29, 2020

V zozname úloh boli pridané ikonky, ktoré znázorňujú, v akom stave je userovo riešenie danej úlohy. Zvýši to prehľadnosť, hneď vidím, čo som už odovzdal, kde mi chýba popis a pod.

image

Každá ikonka má tooltip, ktorý povie userovi viac - "popis: neodovzdaný" a pod.


This change is Reviewable

@codecov
Copy link

codecov bot commented Apr 29, 2020

Codecov Report

Merging #1331 into master will increase coverage by 0.01%.
The diff coverage is 78.57%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #1331      +/-   ##
==========================================
+ Coverage   72.39%   72.40%   +0.01%     
==========================================
  Files         129      129              
  Lines        5433     5447      +14     
  Branches      683      686       +3     
==========================================
+ Hits         3933     3944      +11     
- Misses       1373     1375       +2     
- Partials      127      128       +1     
Impacted Files Coverage Δ
trojsten/contests/templatetags/statements_parts.py 87.80% <77.77%> (-2.83%) ⬇️
trojsten/results/helpers.py 40.97% <80.00%> (+1.40%) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 647fd21...de49e33. Read the comment docs.

Comment on lines 464 to 466
.task-status-wrong {
color: #f39c12;
}
Copy link
Member

Choose a reason for hiding this comment

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

Ak tato farba nie je definovana niekde v nasich design dococh, tak do nej pridaj trocha cervenej / uber zelenu. Ak mas protanopiu alebo nejaku podobnu farboslepost tak je to nerozoznatelne od task-status-ok ak ich nemas hned pri sebe.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Mám zmeniť brand-warning na niečo viac odlišné, keďže to vyplýva z toho?

Copy link
Contributor

Choose a reason for hiding this comment

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

Vieme mat rozne ikonky pre rozne stavy?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Dalo by sa to, ale výber glyphicons je dosť slabý :/

Copy link
Contributor

Choose a reason for hiding this comment

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

Uprimne tie ikonky a farby mi toho moc nehovoria ked sa len tak na to pozrem. Co keby sme to rozdelili na 2 stlpce - stav popisu a stav programu? A potom mali -ok-circle ak je vsetko vpohode, -time ak sa caka na opravenie/sa testuje, -ban-circle ak nie je odovzdany, -remove-circle ak je program nedostal OK? Kazdy stav by zaroven mohol mat aj farbu, ale ikonka aj bez farby by definovala stav, cize by ani nebol problem s farboslepostou.

@gardenerik gardenerik requested a review from black3r April 29, 2020 13:47
@gardenerik
Copy link
Contributor Author

Čo sa týka tých ikoniek, niečo takéto by sa dalo teda spraviť.
image

Teda, ešte nie som úplne stotožnený so zobrazením "neodovzdáva sa", aktuálne "--" (-minus). Takže som otvorený vašim návrhom.

@mhozza
Copy link
Contributor

mhozza commented Jun 14, 2020

Vyzera to super! S tym neodovzdavanim, bud vycentrovana siva horizontalna ciara (pomocou hr), alebo "disabled" N/A.

mhozza
mhozza previously approved these changes Jun 14, 2020
show_description_col = False
for task in tasks:
if task.description_points > 0:
show_description_col = True
Copy link
Contributor

Choose a reason for hiding this comment

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

Toto sa da nahradit idiomatickejsim:
show_description_col = any(map(lambda t: t.description_points >0, tasks))

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.

5 participants