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

Ikonky zobraziujúce stav riešenia. #1331

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

Ikonky zobraziujúce stav riešenia. #1331

wants to merge 7 commits into from

Conversation

gardenerik
Copy link
Collaborator

@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.

css/src/base.less Outdated Show resolved Hide resolved
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
Collaborator 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
Member

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
Collaborator 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
Member

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
Collaborator 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
Member

mhozza commented Jun 14, 2020

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

show_description_col = False
for task in tasks:
if task.description_points > 0:
show_description_col = True
Copy link
Member

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.

4 participants