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.

Sign up for GitHub

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

Jump to bottom

Improve admin budgets form layout #4370

Merged
merged 3 commits into from
Feb 23, 2021
Merged

Improve admin budgets form layout #4370

merged 3 commits into from
Feb 23, 2021

Conversation

javierm
Copy link
Member

@javierm javierm commented Feb 19, 2021 •
edited
Loading

References

Visual changes

Before these changes

Parts of the form are not aligned and the form is not divided in fieldsets

After these changes

The form is divided in fieldset and its components are aligned

@javierm javierm self-assigned this Feb 19, 2021
@javierm javierm added this to Reviewing in Consul Democracy via automation Feb 19, 2021
@javierm javierm changed the title Improve admin budgets form structure Improve admin budgets form layout Feb 19, 2021
@javierm javierm mentioned this pull request Feb 19, 2021
Merged
@javierm javierm force-pushed the admin_budget_form branch 4 times, most recently from 1245d1d to 883da5a Compare February 20, 2021 14:02
@javierm javierm added the UX label Feb 21, 2021
Consul Democracy automation moved this from Reviewing to Testing Feb 23, 2021
@javierm javierm merged commit 8a3f03c into master Feb 23, 2021
Consul Democracy automation moved this from Testing to Release 1.3.0 Feb 23, 2021
@javierm javierm deleted the admin_budget_form branch February 23, 2021 15:24
@javierm javierm added Admin and removed Budgets labels Mar 11, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@taitus taitus taitus approved these changes

Assignees

@javierm javierm

Labels
Admin UX
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
3 participants
@javierm @taitus @decabeza