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

[BUG] Display issue #118

Closed
EVOTk opened this issue Aug 4, 2021 · 4 comments · Fixed by #121
Closed

[BUG] Display issue #118

EVOTk opened this issue Aug 4, 2021 · 4 comments · Fixed by #121
Assignees
Labels
🐛 Bug [ISSUE] Ticket describing something that isn't working

Comments

@EVOTk
Copy link
Contributor

EVOTk commented Aug 4, 2021

Hello,

I currently have two concerns,

1- when I click on the "Update Configuration" key, the page that appears is white, you have to choose a menu to have a display! Is it not possible to have the Configuration menu displayed directly by default?

2021-08-04 22_38_57-Window

2021-08-04 22_38_48-Window

2 - The "backup-note" does not move when you scroll, so it stays in front of the menus.

2021-08-04 22_36_00-Window

2021-08-04 22_36_10-Window

I'm use Firefox, Dashy 1.5.1

Thx

@EVOTk EVOTk added the 🐛 Bug [ISSUE] Ticket describing something that isn't working label Aug 4, 2021
@Lissy93
Copy link
Owner

Lissy93 commented Aug 4, 2021

Thank you- both of those do appear to be bugs. I will work on a fix for them both tomorrow evening :)
I've seen the blank page in the config menu before, but it then disappeared for me, and I have been unable to recreate it since. Did you do anything specific in order for it to be triggered?

@EVOTk
Copy link
Contributor Author

EVOTk commented Aug 4, 2021

Did you do anything specific in order for it to be triggered?

On my installation, it is present all the time now when I open the menu (but I will not say since when).
Whether with Firefox or Chrome (up to date), and even if I have emptied my cache / history, the bug is still present.

I have a simple use of Dashy, nothing exotic: one user, some services, and healtcheck.

In the console, I have no error when I open the Update Configuration page, but when I leave the Configuration page I have this:

TypeError: e is undefined
    findIndexTab vue-material-tabs.esm.js:992
    findIndexTab vue-material-tabs.esm.js:992
    activeTabItem vue-material-tabs.esm.js:949
    disableTabItem vue-material-tabs.esm.js:961
    unRegister vue-material-tabs.esm.js:916
    beforeDestroy vue-material-tabs.esm.js:1097
    VueJS 31
    startTransitionLeave index.js:1
    close index.js:1
    toggle index.js:1
    onOverlayClick index.js:1
    click index.js:1
    VueJS 22
    startTransitionEnter index.js:1
    open index.js:1
    VueJS 9
    open index.js:1
    toggle index.js:1
    onToggle index.js:1
    VueJS 2
    show index.js:1
    show index.js:1
    showEditor ConfigContainer.vue:176
vue.runtime.esm.js:1897:12
    VueJS 34
    startTransitionLeave index.js:1
    close index.js:1
    toggle index.js:1
    onOverlayClick index.js:1
    click index.js:1
    VueJS 22
    startTransitionEnter index.js:1
    open index.js:1
    VueJS 9
    open index.js:1
    toggle index.js:1
    onToggle index.js:1
    VueJS 2
    show index.js:1
    show index.js:1
    showEditor ConfigContainer.vue:176

@Lissy93
Copy link
Owner

Lissy93 commented Aug 5, 2021

That info was really helpful, found the issue and will merge it, along with the fix for the backup-note positioning, as part of #121 in V1.5.3 soon :)

Lissy93 added a commit that referenced this issue Aug 5, 2021
@EVOTk
Copy link
Contributor Author

EVOTk commented Aug 6, 2021

Works great for me! Thank you

asterling8516 pushed a commit to asterling8516/dashy that referenced this issue Nov 23, 2023
Closes Lissy93#118

Signed-off-by: Bjorn Lammers <walkxnl@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🐛 Bug [ISSUE] Ticket describing something that isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants