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

Adding "Hide the used Port"-Option in HomebridgeStatusWidget #572

Merged
merged 9 commits into from
Mar 3, 2020
Merged

Adding "Hide the used Port"-Option in HomebridgeStatusWidget #572

merged 9 commits into from
Mar 3, 2020

Conversation

Staubgeborener
Copy link
Contributor

@Staubgeborener Staubgeborener commented Feb 29, 2020

This PR is a response to #558.

The used port is no longer displayed (on purpose) within the current version. The wrong spelling "running" was already fix'd. But I believe that it should be left open to the community whether they want to display the port or not. I mean, at least one person (also me) misses it. So i added an option "Hide the used Port" for the HomebridgeStatusWidget. You don't want the port? Check the box. You want the port? Uncheck it.

english_running
english_runningport
english_settings

I have also prepared all language files. But one or two lines have to be translated again. Not a big deal.
P.S.: In this PR is also a corrected grammatical mistake in de.json.

@Staubgeborener Staubgeborener changed the title Adding "Hide Port"-Option in HomebridgeStatusWidget Adding "Hide the used Port"-Option in HomebridgeStatusWidget Feb 29, 2020
@oznu
Copy link
Member

oznu commented Mar 1, 2020

Thanks; I'll check this out soon.

We can probably get the translations for "running on port" from an older commit: https://github.com/oznu/homebridge-config-ui-x/tree/140858974f48034ae64cb99aa3cc570fbef26d54/ui/src/i18n

@Staubgeborener
Copy link
Contributor Author

Staubgeborener commented Mar 1, 2020

Good idea.
I used the old commit for the translation and also considered the recent changes (PR #573, PR #571 and PR #568). But there seems to be a conflict with ru.json.

@Caribsky
Copy link
Contributor

Caribsky commented Mar 1, 2020

Hi, @Staubgeborener
Regarding ru.json. All merges are correct.

ui/src/i18n/ru.json Outdated Show resolved Hide resolved
@oznu
Copy link
Member

oznu commented Mar 3, 2020

Thanks @Staubgeborener and @Caribsky

@oznu oznu merged commit f694c9f into homebridge:master Mar 3, 2020
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.

3 participants