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

Some part of the UI shown even should be hidden... #92

Closed
FredM67 opened this issue Oct 14, 2021 · 0 comments
Closed

Some part of the UI shown even should be hidden... #92

FredM67 opened this issue Oct 14, 2021 · 0 comments

Comments

@FredM67
Copy link
Contributor

FredM67 commented Oct 14, 2021

In the home.html, both groups "Latest Data" and "Control" should be hidden when status.isWifiClient is false.

I think, the data-bind="visible should be on the parent element. It seems the style.display property overwrite the data-bind.

@FredM67 FredM67 changed the title Some part of the UI shown even should hidden... Some part of the UI shown even should be hidden... Oct 14, 2021
FredM67 added a commit to FredM67/EmonESP that referenced this issue Oct 22, 2021
FredM67 added a commit to FredM67/EmonESP that referenced this issue Oct 22, 2021
commit 49d81ae
Author: Frédéric Metrich <45318189+FredM67@users.noreply.github.com>
Date:   Fri Oct 22 22:33:39 2021 +0200

    Update home.html

    Fix issue openenergymonitor#92
FredM67 added a commit to FredM67/EmonESP that referenced this issue Dec 3, 2021
commit 5ef0042
Author: Frédéric Metrich <45318189+FredM67@users.noreply.github.com>
Date:   Fri Dec 3 18:50:50 2021 +0100

    Add time support with TZ and DST

commit de0f047
Author: Frédéric Metrich <45318189+FredM67@users.noreply.github.com>
Date:   Thu Dec 2 20:28:25 2021 +0100

    Remove

commit be22475
Author: Frédéric Metrich <45318189+FredM67@users.noreply.github.com>
Date:   Thu Dec 2 15:47:13 2021 +0100

    Add submodule

commit ee2cbc3
Merge: 1c1a7c4 0af0551
Author: Frédéric Metrich <45318189+FredM67@users.noreply.github.com>
Date:   Tue Oct 26 20:02:19 2021 +0200

    Merge branch 'openenergymonitor:master' into master

commit 0af0551
Merge: 596572a a32c2d6
Author: Glyn Hudson <glynhudson@users.noreply.github.com>
Date:   Tue Oct 26 18:49:43 2021 +0100

    Merge pull request openenergymonitor#97 from FredM67/bugfix-95

    Control mode saved when changed

commit 1c1a7c4
Author: Frédéric Metrich <45318189+FredM67@users.noreply.github.com>
Date:   Sat Oct 23 00:33:58 2021 +0200

    Update .travis.yml (#1)

commit a32c2d6
Author: Frédéric Metrich <45318189+FredM67@users.noreply.github.com>
Date:   Fri Oct 22 22:47:46 2021 +0200

    Control mode saved when changed

    Fix issue openenergymonitor#95 and other stuff.

commit 596572a
Merge: 6e0b89c 49d81ae
Author: Glyn Hudson <glynhudson@users.noreply.github.com>
Date:   Fri Oct 22 21:39:30 2021 +0100

    Merge pull request openenergymonitor#96 from FredM67/bugfix-92

    Update home.html

commit 49d81ae
Author: Frédéric Metrich <45318189+FredM67@users.noreply.github.com>
Date:   Fri Oct 22 22:33:39 2021 +0200

    Update home.html

    Fix issue openenergymonitor#92
FredM67 pushed a commit to FredM67/EmonESP that referenced this issue Feb 19, 2022
commit c62d144
Author: Frederic Metrich <frederic.metrich@siemens.com>
Date:   Sun Feb 20 00:01:57 2022 +0100

    clang-format

commit 1a26729
Author: Frederic Metrich <frederic.metrich@siemens.com>
Date:   Sat Feb 19 23:55:33 2022 +0100

    clang-tidy

commit 5ef0042
Author: Frédéric Metrich <45318189+FredM67@users.noreply.github.com>
Date:   Fri Dec 3 18:50:50 2021 +0100

    Add time support with TZ and DST

commit de0f047
Author: Frédéric Metrich <45318189+FredM67@users.noreply.github.com>
Date:   Thu Dec 2 20:28:25 2021 +0100

    Remove

commit be22475
Author: Frédéric Metrich <45318189+FredM67@users.noreply.github.com>
Date:   Thu Dec 2 15:47:13 2021 +0100

    Add submodule

commit ee2cbc3
Merge: 1c1a7c4 0af0551
Author: Frédéric Metrich <45318189+FredM67@users.noreply.github.com>
Date:   Tue Oct 26 20:02:19 2021 +0200

    Merge branch 'openenergymonitor:master' into master

commit 0af0551
Merge: 596572a a32c2d6
Author: Glyn Hudson <glynhudson@users.noreply.github.com>
Date:   Tue Oct 26 18:49:43 2021 +0100

    Merge pull request openenergymonitor#97 from FredM67/bugfix-95

    Control mode saved when changed

commit 1c1a7c4
Author: Frédéric Metrich <45318189+FredM67@users.noreply.github.com>
Date:   Sat Oct 23 00:33:58 2021 +0200

    Update .travis.yml (#1)

commit a32c2d6
Author: Frédéric Metrich <45318189+FredM67@users.noreply.github.com>
Date:   Fri Oct 22 22:47:46 2021 +0200

    Control mode saved when changed

    Fix issue openenergymonitor#95 and other stuff.

commit 596572a
Merge: 6e0b89c 49d81ae
Author: Glyn Hudson <glynhudson@users.noreply.github.com>
Date:   Fri Oct 22 21:39:30 2021 +0100

    Merge pull request openenergymonitor#96 from FredM67/bugfix-92

    Update home.html

commit 49d81ae
Author: Frédéric Metrich <45318189+FredM67@users.noreply.github.com>
Date:   Fri Oct 22 22:33:39 2021 +0200

    Update home.html

    Fix issue openenergymonitor#92
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

No branches or pull requests

2 participants