-
Notifications
You must be signed in to change notification settings - Fork 43
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
205 filter publications UI #2301
Conversation
9b2b5ae
to
4ffca44
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It doesn't necessarily have to be part of this PR, but we should at least do it as part of the Trello ticket to create a non-legaqy version of the integration tests in test/integration/legacy_root_overview_test.rb
app/views/root/_filter.html.erb
Outdated
} %> | ||
|
||
<%= render "govuk_publishing_components/components/select", { | ||
id: "format_filter", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Maybe now would be a good time to rename the various usages of the term "format" with "content type" in the new (non-legacy) code.
4536ae5
to
c5d653c
Compare
With this PR, we should probably port all the tests in |
eb83beb
to
1d7a41a
Compare
995eb29
to
b23a112
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
A couple of "format" references left that we should probably rename to "content_type"; other than that, LGTM—but someone else will have to approve, since I pushed one of the commits on this PR.
f1623b4
to
a998a8f
Compare
- Add stylesheet for Publications page - Add publishing components to filter form - Add new tests and update existing tests
a998a8f
to
a814f65
Compare
Trello
This work primarily adds the UI to the filter section of the publications page into the left hand column (see screenshot below).
In addition to that there is some logic to pre-populate fields where required which could probably do with a bit more discussion form a Rails perspective.