Skip to content
This repository has been archived by the owner on Nov 30, 2023. It is now read-only.

Fix schedule viewer ui #459

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

Fix schedule viewer ui #459

wants to merge 2 commits into from

Commits on Nov 21, 2018

  1. Fix bottombar's visibility 

    The text element just under the schedule's SVG was partially hidden when
the bottombar's height was set to 175px. I tried to remove the height
property or to set it to auto. This was working great under Chrome, but
Firefox seems to have problem calculating the content element's height...
unless the debug pane was opened (???)

So, for now, fix the height by setting a proper value (198px).

Fix a comment at the same time for readability.

Signed-off-by: Alexandre Demers <[email protected]>
    @Oxalin
    Oxalin committed Nov 21, 2018
    Configuration menu
    Copy the full SHA
    ef1193b View commit details
    Browse the repository at this point in the history

  2. Fix Date input field to display date as yyyyMMdd 

    The yyyy, MM and dd values were added one to the other under
setStartDate(). 2018, 11, 21 would end up being 2050 instead of 20181121.

Force the variables to be treated as strings.

Signed-off-by: Alexandre Demers <[email protected]>
    @Oxalin
    Oxalin committed Nov 21, 2018
    Configuration menu
    Copy the full SHA
    38a486b View commit details
    Browse the repository at this point in the history