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

UI update #1

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

UI update #1

wants to merge 100 commits into from

Conversation

AlexFreik
Copy link

@AlexFreik AlexFreik commented Jul 13, 2023

Important: there are so many additions and deletions because I autoformated the files in this commit: 5f350f3
Before formating it was a big mess of html, and there were a few missed tags, or wrong tags, which I fixed in: 23caff6

From original files only the html/settings.html file was changed functionally. All other html files were just autoformated, and I didn't touch them after this.

I dockerized the project, so anyone can run the whole thing with just one command $ docker compose up. Right now it is used only for development, and the old way of running with install-nginx.sh works the same.

Here are UI changes inhtml/settings.html file:

  1. Basic styling, so that it is not black & white.
  2. When user adds new destination, the server response appears in the bottom right corner in a queue like manner instead of oppening a new tab:
    Screenshot 2023-07-13 at 15 29 18
  3. All streams can have nicknames, so it is easier to add new destinations (they will only appear on the UI, and do not interfeer with backend):
    Screenshot 2023-07-13 at 15 35 57
    One can set the names in the table at the bottom of settings page:
    Screenshot 2023-07-13 at 15 32 14
  4. Dropdown with Server URL default values, so it is faster to add YouTube or Facebook streams:
    Screenshot 2023-07-13 at 15 33 28

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

Successfully merging this pull request may close these issues.

1 participant