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

Upgrade govuk_admin_template, tweak application table UI #91

Merged
merged 4 commits into from Apr 21, 2015

Conversation

@fofr
Copy link
Contributor

@fofr fofr commented Apr 21, 2015

Upgrade govuk_admin_template

  • Upgrade to Bootstrap 3.3.2
  • Include Google Analytics

Open filtered application link on submit

After filtering a list of applications, a common action is to hit enter to load the top item showing.

The admin template supports this using the js-open-on-submit class, which indicates which link in a row to open when the filter form is submitted.

Use links not labels for releases

Links are obviously clickable and come with visited link styles

screen shot 2015-04-21 at 17 03 26

cc @jamiecobbett

fofr added 4 commits Apr 21, 2015
* Upgrade to Bootstrap 3.3.2
* Include Google Analytics
After filtering a list of applications, a common action is to hit enter
to load the top item showing.

The admin template supports this using the `js-open-on-submit` class,
which indicates which link in a row to open when the filter form is
submitted.
Make it clearer that the release is clickable. Visited link styles are
a bonus.
“In sync?” would sometimes collapse onto two lines.
jamiecobbett added a commit that referenced this pull request Apr 21, 2015
Upgrade govuk_admin_template, tweak application table UI
@jamiecobbett jamiecobbett merged commit 4ee220b into master Apr 21, 2015
1 check passed
1 check passed
default "Build #18 succeeded on Jenkins"
Details
@jamiecobbett jamiecobbett deleted the enter-on-filter branch Apr 21, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

2 participants
You can’t perform that action at this time.