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

Agents configuration tab redesign #310

Merged
merged 16 commits into from Mar 13, 2018
Merged

Agents configuration tab redesign #310

merged 16 commits into from Mar 13, 2018

Conversation

JuanjiJG
Copy link
Contributor

Hello all,

This PR brings a new redesign to the app. This time is for the Agents configuration tab.

Description

This redesign is based on the same made for the Manager configuration tab. With this UI revamp, some modifications were made to the side nav cards and were applied to both tabs, providing consistency between them.

For example, the fa-eye icon was removed, and replaced by a bold-text active element indicator. When the user hovers the title, the font will turn bold, and also a tooltip will appear, indicating the possible actions.

The current group and sync status indicators were replaced by the new information ribbon, the same that we can see on other app sections.

In addition to this, this PR fixes some minor bugs on the UI, and also removes old CSS classes, continuing with the huge work done with the HTML & CSS removal and cleanup process.

Sample screenshot

rediseno

nogroup

This redesign is one more step into a better app design, achieving consistency between pages and a better overall look.

Best regards,
Juanjo

@havidarou
Copy link
Member

LGTM!

@havidarou havidarou merged commit 2891b14 into 3.2 Mar 13, 2018
@havidarou havidarou deleted the 3.2-dev-agent-conf branch March 13, 2018 10:39
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.

None yet

2 participants