Again related to #2089309: Create job for multiple target languages.

Right now the UX when having multiple jobs to check out is quite bad. The only visible indication beside the initial status message is that the button shows a ".. and continue".

As part of improving working with multiple target languages, this issue is about improving the checkout UI, adding a few features, making others smaller/hiding them by default.

Things we discussed so far:

* Showing a progress element that should how many jobs there are in total and how many are still left. Somewhere on top, like a checkout progress.
* A checkbox that allows to attempt to submit all remaining jobs with the settings of the current job. Either when checking that checkbox or always, it would also be nice if we'd show the jobs then (title and source/target language, maybe item/word count)
* Hide the messages details element if there are no messages. I think this used to work but something broke it.
* Collapse the job items by default, show the count in the details title
* Move the suggestions elements out of the job items details and keep it visible by default.

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Berdir created an issue. See original summary.

Berdir’s picture

Berdir’s picture

Issue summary: View changes
Berdir’s picture

This is a work in progress patch, changing a lot of stuff and still incomplete an certainly going to fail a lot of tests:

* Fixed messages fieldset to only show up if there are messages
* Load suggestions button is gone, instead they are already loaded, if there are none, the suggestions fieldset is not shown. If there are, the job items fieldset is collapsed.
* Introduced a JobQueue service to deprecate the tmgmt_redirect_* functions that has a lot more functionality that we need to track processed items and handle bulk submission
* Introduced a JobCheckoutManager to add replacement functions of checkout helper functions that use the new JobQueue instead of the deprecated functions.
* In the progress of updating usages of those deprecated functions, will also need to document that properly and everything ;)
* Added a progress bar to the top of the checkout form, it's not very pretty yet though ;)
* Started implementing bulk submit, it works but does not yet use batch, which means there can be a very wait time.

Was hoping to keep the two patches separate, but that won't work anymore with all the refactoring, so two patches for now, one against 8.x-1.x and one against the other issue.

The last submitted patch, 4: tmgmt-multi-checkout-2914080-4.patch, failed testing. View results
- codesniffer_fixes.patch Interdiff of automated coding standards fixes only.

Berdir’s picture

The last submitted patch, 6: tmgmt-multi-checkout-2914080-6.patch, failed testing. View results
- codesniffer_fixes.patch Interdiff of automated coding standards fixes only.

Berdir’s picture

Fixing some test fails and adding support for using batch.

Berdir’s picture

New patch, this considerably improves the progress bar by re-using the standard progress bar UI. Also added the list of pending job labels and languages in a collapsed details element. And added a few tests.

The last submitted patch, 9: tmgmt-multi-checkout-2914080-9.patch, failed testing. View results

Berdir’s picture

Berdir’s picture

Status: Needs review » Fixed

Committed.

  • Berdir committed 3ce79c1 on 8.x-1.x
    Issue #2914080 by Berdir: Improve checkout UI when using a translation...

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.

badrange’s picture

Am I the only one who have crashed into a bug when requesting translations after this issue was merged? See https://www.drupal.org/project/tmgmt/issues/2951475