Overview

If you have a lot of changes (in this case 10) and one of more of them are unable to be published, there is visible indication why the publishing is failing. The errors ARE displayed, but unless you know to scroll down, you can't see them!

Follow up from #3536127: When errors are found, they are reported, but XB still says changes are published

Proposed resolution

Move the errors up, inside the scrollable container above the list of unpublished changes.

User interface changes

See attached png.

Command icon Show commands

Start within a Git clone of the project using the version control instructions.

Or, if you do not have SSH keys set up on git.drupalcode.org:

Comments

jessebaker created an issue. See original summary.

wim leers’s picture

Issue tags: +JavaScript
libbna’s picture

Assigned: Unassigned » libbna
Issue tags: -JavaScript +JavaScript
StatusFileSize
new25.27 KB

I’ve worked on a potential fix for this issue. Before pushing the changes, I wanted to check if the approach looks good—can I go ahead with it?

callumharrod’s picture

StatusFileSize
new55.02 KB

Adding design for the selective publishing dropdown here:
Selective publishing dropdown with errors

libbna’s picture

Thanks for sharing the design @callumharrod I will work on this on Monday.

libbna’s picture

Assigned: libbna » Unassigned
StatusFileSize
new27.12 KB

Till now I have achieved this much:

I have moved the error reports to the top along with the changes selected section.

publish section

I will be unassigning myself for the time being until further review or feedback is provided.

libbna’s picture

libbna’s picture

Status: Active » Needs work
jessebaker’s picture

Assigned: Unassigned » jessebaker

Assigning to me to remind me to look at this tomorrow morning. Thanks @libbna

  • jessebaker committed a7d1cbe7 on 1.x authored by libbna
    Issue #3536649 by libbna, jessebaker: Move Publishing error reports to...
jessebaker’s picture

Assigned: jessebaker » Unassigned
Status: Needs work » Fixed

Tested, approved and merged! Thank you @libbna!

wim leers’s picture

Thanks for the screenshots — that looks exactly like what I expected/hoped based on the title! 😄

Also crediting @callumharrod for #4 😊

Status: Fixed » Closed (fixed)

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