Commit Graph

9452 Commits

Author SHA1 Message Date
Chris Hill-Scott
414dee7002 Fix tests 2020-02-27 16:29:14 +00:00
karlchillmaid
117c18d3cf Update guidance 2020-02-27 16:27:30 +00:00
karlchillmaid
fdf07eb9c5 Update content 2020-02-27 16:26:15 +00:00
karlchillmaid
1fd0ead7e5 Update formatting 2020-02-27 16:24:56 +00:00
karlchillmaid
6f3cf91f1b Update content so it matches 3302
Update content so it matches the changes originally made in https://github.com/alphagov/notifications-admin/pull/3302
2020-02-27 16:24:13 +00:00
karlchillmaid
c7c24f5f8c Replace upload a letter content
Replace upload a letter content with a more generic description.
2020-02-27 16:22:43 +00:00
karlchillmaid
82f9472440 Update page title and H1 2020-02-27 16:22:00 +00:00
karlchillmaid
a3519bb331 Update placeholder content 2020-02-27 16:21:26 +00:00
karlchillmaid
d92809ac47 Change layout and add placeholder content 2020-02-27 16:20:49 +00:00
Rebecca Law
0a43e91372 Merge pull request #3332 from alphagov/organisation-usage
Add usage stats to organisation page.
2020-02-27 11:57:23 +00:00
Rebecca Law
4b11d776ee Add usage stats to organisation page. 2020-02-26 13:24:18 +00:00
Tom Byers
07e7d98407 Merge pull request #3331 from alphagov/convert-all-links-to-govuk-frontend-2nd-attempt
Convert all links to govuk frontend 2nd attempt
2020-02-25 15:50:54 +00:00
Tom Byers
e6767c4d01 Make links in validation summaries red 2020-02-25 15:32:31 +00:00
Tom Byers
7755c604f4 Merge pull request #3322 from alphagov/remove-unused-files-and-config
Remove unused files and config
2020-02-25 14:30:23 +00:00
Chris Hill-Scott
35be1ce31a Merge pull request #3330 from alphagov/bump-bleach
Bump utils to fix bleach security vulnerability
2020-02-25 11:35:05 +00:00
Chris Hill-Scott
519b340c1a Merge pull request #3328 from alphagov/hide-jobs-from-dashboard
Hide jobs on dashboard for users with uploads page
2020-02-25 11:33:00 +00:00
Tom Byers
7808c55869 Change how link is referenced in user profile test
Co-Authored-By: Katie Smith <klssmith@users.noreply.github.com>
2020-02-25 10:52:03 +00:00
Tom Byers
9f2e1498fc Update links missed in previous work
Includes those in:
- the static 5xx error page
- validation messaging
- guidance page about trial mode
- links used in tests

Issues raised in https://github.com/alphagov/notifications-admin/pull/3303#issuecomment-589572715
2020-02-25 10:52:03 +00:00
Tom Byers
efa892423a Fix issues raised on pull request
Includes:
- make 'remove team member' link, on edit member
  permissions page, destructive
- convert missed links on /features pages
- convert missed links on /using-notify/guidance and sub pages
- give links in browse-lists back their size and
  weight (needed for lists of live and trial
  services on Platform Admin)
- give links on Platform Admin inbound numbers
  page back their size and weight
- update links in JS tests
2020-02-25 10:52:03 +00:00
Tom Byers
16c1c34588 Fix tests broken by changes to HTML 2020-02-25 10:52:03 +00:00
Tom Byers
dc941bc9bc Remove all redundant link CSS
Giving all links the GOVUK Frontend classes, and
the new `govuk-link--destructive` class, means
some styles are already applied.

This strips out those styles.

Note: there's still a good amount of styling, most
of which is to make the focus styles specific to
the space the link is in. These will need
reviewing when GOVUK Frontend is bumped past
version 3 as this brings in new focus styles.
2020-02-25 10:52:03 +00:00
Tom Byers
8996b43496 Remove style for deleted elements
The `.global-cookie-message` and `.footer-nav`
classes were for elements in the govuk_template.
This used to be the base layout for this app but
was changed here:

https://github.com/alphagov/notifications-admin/pull/3198
2020-02-25 10:52:03 +00:00
Tom Byers
2a71cc5481 Convert warning links to govuk-link--destructive
Converts links in the following:
- the page-footer component
- the table component
- the browse-list component
- the notification status, when reporting failures
- validation messaging in the whitelist page
2020-02-25 10:52:03 +00:00
Tom Byers
9a6916ff31 Replace links in navigation
Includes:
- removing all styling of those links
  outside of GOV.UK Frontend (except for a few fixes
  due to their interaction with our design)
- bringing together some duplicate styles into one
  block
- changing how links are marked as selected, now
  they have multiple classes

Fix Sass-lint error in navigation.scss
2020-02-25 10:52:03 +00:00
Tom Byers
58c08ad89b Replace browse-list links 2020-02-25 10:52:03 +00:00
Tom Byers
61afbe7a8f Add styles for destructive links
This colour is used in a few places so worth
making into its own block and classes.

Note: this establishes a
`app/assets/stylesheets/local` folder for any Sass
files whose names clash with files from vendored
libraries.

`_typography.scss` already exists in GOV.UK
Elements Sass.

This will no longer be needed when GOV.UK Elements
is gone and GOV.UK Frontend is at version 3, where
all its folders are in a `govuk` folder.
2020-02-25 10:52:02 +00:00
Tom Byers
34e1eee364 Add font-smoothing as global style
Making all links GOV.UK Frontend styled means some
will sit in content that doesn't use the GOV.UK
Frontend font styles yet.

This applies the font-smoothing that comes with
those styles to all text so links do not look out
of place on browsers that support it.

Note: font-smoothing was part of the GOV.UK
Template styles which have now been removed:

https://github.com/alphagov/govuk_template/blob/master/source/assets/stylesheets/_basic.scss#L68
2020-02-25 10:47:31 +00:00
Tom Byers
346c4aa7f3 Update links in JavaScript 2020-02-25 10:47:30 +00:00
Tom Byers
36ddd52db1 Remove global link styles
All links, that need it, now have the GOVUK
Frontend link class so we no longer need this
CSS.
2020-02-25 10:47:30 +00:00
Tom Byers
ee9f348ce4 Update all links to use GOVUK Frontend style
Includes:
- turning off :visited styles to match existing
  design
- swapping heading classes used to make links bold
  for the GOVUK Frontend bold override class
- adding visually hidden text to some links to
  make them work when isolated from their context

We may need to revisit whether some links, such as
those for documentation and features, may benefit
from having some indication that their target has
been visited.
2020-02-25 10:47:24 +00:00
Chris Hill-Scott
55ebecbca5 Bump utils to fix bleach security vulnerability 2020-02-25 10:40:11 +00:00
David McDonald
f0c4dd9c99 Merge pull request #3327 from alphagov/cruft
Remove things from readme that are no longer relevant
2020-02-25 10:28:12 +00:00
Chris Hill-Scott
b12df3edbf Merge pull request #3316 from alphagov/summarise-scheduled-jobs-on-dashboard
Summarise scheduled jobs on the dashboard
2020-02-25 10:15:22 +00:00
Chris Hill-Scott
8914755b1e Hide jobs on dashboard for users with uploads page
They can see them there instead. We can tidy this up later once we’ve
migrated everyone onto having the permission.
2020-02-24 18:06:22 +00:00
David McDonald
21c63926b5 Remove things from readme that are no longer relevant
The requirements link has been broken for ages
We no longer use coveralls
2020-02-24 17:31:29 +00:00
Chris Hill-Scott
f2f531a7e6 Summarise scheduled jobs on the dashboard
Scheduled jobs push everything else on the dashboard down, which makes
them very prominent. This is exacerbated by people scheduling more jobs
simultaneously than we expected when we originally designed the feature.

We also want to remove all jobs from the dashboard, in favour of putting
them on the uploads page.

So this commit replaces them with one of our new dashboard banners (used
for received text messages in returned letters) which summarises:
- how many scheduled jobs you have
- when the first one is going out (i.e. how long you have to stop it, if
  you need to)
2020-02-24 17:20:12 +00:00
Chris Hill-Scott
7157b1cee8 Merge pull request #3306 from alphagov/scheduled-on-uploads
Show scheduled jobs on page one of the uploads page
2020-02-24 16:48:32 +00:00
Chris Hill-Scott
8345bbe0b3 Use pre-defined mock from conftest
Rather than manually writing it.
2020-02-24 16:38:10 +00:00
Chris Hill-Scott
3dab9c2915 Refactor to make it logic clearer
The logic around what gets shown on the uploads page was hard to follow.
This commit makes three changes to hopefully make it easier:
- remove the passing-around of a string containing the contents of a
  rendered partial
- encapsulate everything in one `show_scheduled_jobs` variable, rather
  than mixing between `scheduled_jobs` and
  `current_service.scheduled_jobs`
- adds a comment to explain why we still render `_jobs.html` even when
  we know we have no jobs
2020-02-24 16:38:09 +00:00
Chris Hill-Scott
c20ec82cd2 Show scheduled jobs on page one of the uploads page
This is the same thing we do for caseworking users who don’t have the
dashboard. Since we’re going to summarise scheduled jobs on the
dashboard instead of listing them they need to be listed here instead
(which is where we’ll link to from the dashboard).

Design of this will probably evolve as we work out how to style single
letter uploads and letter jobs, but that’s OK for now because no-one
has the uploads page at the moment.
2020-02-24 16:38:07 +00:00
karlchillmaid
e13dba3763 Merge pull request #3324 from alphagov/change-forgot-to-forgotten
Change forgot to forgotten
2020-02-24 12:45:17 +00:00
Tom Byers
b128001774 Merge pull request #3323 from alphagov/revert-3303-convert-all-links-to-govuk-frontend
Revert "Convert all links to govuk frontend"
2020-02-24 12:37:52 +00:00
karlchillmaid
6c4013d207 Change 'forgot' to 'forgotten' 2020-02-24 12:00:26 +00:00
karlchillmaid
a33892bded Change 'forgot' to 'forgotten' 2020-02-24 11:59:43 +00:00
karlchillmaid
a51cc5d4b5 Change 'forgot' to 'forgotten' 2020-02-24 11:59:07 +00:00
karlchillmaid
11808cbf88 Change 'forgot' to 'forgotten' 2020-02-24 11:57:48 +00:00
Tom Byers
5b306dde4d Revert "Convert all links to govuk frontend" 2020-02-24 11:56:38 +00:00
Tom Byers
ae1b2d4c63 Merge pull request #3303 from alphagov/convert-all-links-to-govuk-frontend
Convert all links to govuk frontend
2020-02-24 10:30:54 +00:00
Tom Byers
97172cca30 Merge branch 'master' into convert-all-links-to-govuk-frontend 2020-02-24 10:12:58 +00:00
karlchillmaid
95b41b6bfa Merge pull request #3317 from alphagov/remove-links-from-get-started
Remove links to templates and permissions
2020-02-24 10:04:11 +00:00