-
-
Notifications
You must be signed in to change notification settings - Fork 393
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
Refactor PR #957 #1234
Refactor PR #957 #1234
Conversation
Hello @Paulchen-Panther 👋 I'm your friendly neighborhood bot and would like to say thank you for So that you and other users can test your changes more quickly, If you make changes to your PR, i create a new link to your workflow artifacts. Best regards, |
Here is your new link to your workflow artifacts. |
Here is your new link to your workflow artifacts. |
Here is your new link to your workflow artifacts. |
Here is your new link to your workflow artifacts. |
Here is your new link to your workflow artifacts. |
This pull request fixes 1 alert when merging 113f821 into 71e34a6 - view on LGTM.com fixed alerts:
|
Here is your new link to your workflow artifacts. |
This pull request introduces 1 alert and fixes 10 when merging c8acd2c into 71e34a6 - view on LGTM.com new alerts:
fixed alerts:
|
Here is your new link to your workflow artifacts. |
Here is your new link to your workflow artifacts. |
This pull request introduces 1 alert and fixes 9 when merging 9a169e5 into 71e34a6 - view on LGTM.com new alerts:
fixed alerts:
|
instance panel always visible
Here is your new link to your workflow artifacts. |
This pull request fixes 9 alerts when merging 61c3c8b into 71e34a6 - view on LGTM.com fixed alerts:
|
restore main_nav position after switching Destop/Mobile
Here is your new link to your workflow artifacts. |
Here is your new link to your workflow artifacts. |
Here is your new link to your workflow artifacts. |
Here is your new link to your workflow artifacts. |
Here is your new link to your workflow artifacts. |
This pull request fixes 9 alerts when merging 1b783cd into 71e34a6 - view on LGTM.com fixed alerts:
|
Summary
Since the current documentation and the forum have already been redesigned, it is time to adapt the main component. As a bonus, feature request #1136 has been implemented.
A BIG thanks goes to our graphic designer @mkcologne .
What kind of change does this PR introduce? (check at least one)
ReDesign WebUI to reflect the current design
If changing the UI of web configuration, please provide the before/after screenshot:
Example screenshots of old WebUI vs. new WebUI
https://screenshotcomparison.com/comparison/13460/
Does this PR introduce a breaking change? (check one)
If yes, please describe the impact and migration path for existing setups:
The PR fulfills these requirements:
Fixes: #xxx[,#xxx]
, where "xxx" is the issue number)If adding a new feature, the PR's description includes:
PLEASE DON'T FORGET TO ADD YOUR CHANGES TO CHANGELOG.MD
To avoid wasting your time, it's best to open a feature request issue first and wait for approval before working on it.
Other information:
Closes #1136