italia / docs-italia-convertitore-web

Other
4 stars 1 forks source link

Add support for converter options #21

Closed yakky closed 6 years ago

yakky commented 6 years ago

Adds support for converti json options

screenshot_20180725_123926

yakky commented 6 years ago

@francescozaia @danse @atorin cosa ne pensate dal punto di vista della UI?

codecov[bot] commented 6 years ago

Codecov Report

Merging #21 into master will increase coverage by 14.27%. The diff coverage is 87.5%.

Impacted file tree graph

@@             Coverage Diff             @@
##           master      #21       +/-   ##
===========================================
+ Coverage   42.01%   56.29%   +14.27%     
===========================================
  Files           6        6               
  Lines         119      135       +16     
  Branches        7        8        +1     
===========================================
+ Hits           50       76       +26     
+ Misses         69       58       -11     
- Partials        0        1        +1
Impacted Files Coverage Δ
docs_italia_convertitore_web/views.py 0% <0%> (ø) :arrow_up:
docs_italia_convertitore_web/forms.py 100% <100%> (+100%) :arrow_up:
docs_italia_convertitore_web/tasks.py 78.26% <85.71%> (+7.04%) :arrow_up:

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 795f4d4...bd6cc86. Read the comment docs.

yakky commented 6 years ago

@danse riguardo ai tooltip non sono sicuro che i tooltip di bootstrap italia permettano di includere link @francescozaia ci puoi dare support? Elaboro un secondo: per implementare la "i" come suggerito da francesco, pensavo di usare i tooltip di boostrap italia (https://italia.github.io/bootstrap-italia/docs/componenti/tooltip/#markup) che permettono anche markup complesso, però essendo sull'hover dell'elemento, appena l'utente si muove sul tooltip, questo si chiude E' sufficiente mettere un delay abbastanza lungo (come documentato qui https://getbootstrap.com/docs/4.1/components/tooltips/#options )?

francescozaia commented 6 years ago

@yakky quello che cerchi è in realtà il Popover. 🙂 Detto questo, ricordiamoci di non aggiungere ulteriore lavoro. Si era detto di lasciare le opzioni fuori da questa prima versione. Se ormai volete includerle ok, ma a patto che:

yakky commented 6 years ago

@francescozaia no, è poco lavoro e non ci sto investendo particolarmente

yakky commented 6 years ago

@danse @francescozaia aggiornato codice e screenshot

danse commented 6 years ago

grande! visualmente io però metterei:

se siete d'accordo. è chiaro quello che voglio dire? prima la box, poi il testo, poi l'icona info

yakky commented 6 years ago

@danse fatto e screen aggiornato

danse commented 6 years ago

perfetto! :+1: