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

feat(frontend): allow managing multiple nodes on one page #66

Merged
merged 18 commits into from
Dec 29, 2021

Conversation

ravenclaw900
Copy link
Owner

No description provided.

@ravenclaw900 ravenclaw900 changed the title build(backend): allow building backend-only frontend: allow managing multiple nodes on one page Dec 8, 2021
@ravenclaw900 ravenclaw900 added the enhancement New feature or request label Dec 8, 2021
@ravenclaw900 ravenclaw900 linked an issue Dec 8, 2021 that may be closed by this pull request
@ravenclaw900
Copy link
Owner Author

ravenclaw900 commented Dec 18, 2021

Well, the viewing of different nodes works, but, for instance, the update message pushes the node selection box out of the way, which pushes the light/dark mode button out. How should we go about fixing this? I'm thinking something like a "messages" icon, which will reveal the update and any other messages, and maybe a "settings" icon for the node selection box, and possibly themes once those get implemented.

config.toml Outdated Show resolved Hide resolved
@MichaIng
Copy link
Collaborator

A notification button/icon seems a good idea. In theory, it could be also shown conditionally, i.e. either if there are multiple notifications or when not everything fits into the viewport with a single notification expanded.

@ravenclaw900
Copy link
Owner Author

Alright, notification button added. If that looks good, I'm going to merge and release.

@MichaIng
Copy link
Collaborator

I'll do some testing later today, many thanks for implementing 👍.

@ravenclaw900 ravenclaw900 changed the title frontend: allow managing multiple nodes on one page feat(frontend): allow managing multiple nodes on one page Dec 29, 2021
@ravenclaw900 ravenclaw900 merged commit c2b6953 into main Dec 29, 2021
@ravenclaw900 ravenclaw900 deleted the backend-only branch December 29, 2021 23:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Request - Add multiple node dashboard
2 participants