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

Change server name to avoid confusion #4

Merged
merged 2 commits into from
Oct 23, 2018
Merged

Change server name to avoid confusion #4

merged 2 commits into from
Oct 23, 2018

Conversation

timmo001
Copy link
Member

@timmo001 timmo001 commented Oct 9, 2018

Proposed Changes

The current server name can get a bit confusing since it changes the title of the motionEye page.

The only way to know the tab is for motionEye if from the icon:

tabs

This changes the server name to motionEye | Hass.io so the title is more distinguishable.

@addons-assistant
Copy link

💖 Thanks for opening this pull request! 💖 If your PR gets accepted and merged in, we will invite you to the project 🎉

callelj
callelj previously approved these changes Oct 18, 2018
@frenck
Copy link
Member

frenck commented Oct 18, 2018

I do not agree on the change since that would display motionEye twice in the header, which just look ugly as freaking h*ll as well.

Secondly, the icon differs...?

@frenck frenck added the Idea label Oct 18, 2018
@frenck frenck merged commit 81c19c2 into hassio-addons:master Oct 23, 2018
@addons-assistant
Copy link

Congrats on merging your first pull request! 🎉🎉🎉

@timmo001 timmo001 deleted the patch-1 branch October 23, 2018 21:15
@addons-assistant
Copy link

This thread has been automatically locked because it has not had recent activity. Please open a new issue for related bugs and link to relevant comments in this thread.

@addons-assistant addons-assistant bot locked as resolved and limited conversation to collaborators Nov 22, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants