Steps to reproduce

Install Drupal 10 with Claro as the admin theme.
Install scheduled_transitions module.
Enable workflows for the blocks.
/admin/config/workflow/workflows/manage/editorial

Visit /admin/content/block/1/scheduled-transitions and add transition.
You will see for the first transition the size of Schedule transition button is smaller as compared to second transition. Adding screenshot for reference. Also there is a z-index difference in both transitions.

Issue fork drupal-3383637

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

Gauravvvv created an issue. See original summary.

Harish1688’s picture

looking this issue

Harish1688’s picture

StatusFileSize
new164.98 KB
new185.94 KB

Hi,

Looking the issue schedule transition button size is different for first transition and for second transition, I have added second and third transition but button found the transition button size same as in all states. attached a reference image for this.

can you brief the more details, so i can reproduce it.

Testing Steps:-
1. Drupal Set 10 with claro as the admin theme.
2. For scheduled transitions module follow the same steps mention in the ticket.
3. On node page (node/1/scheduled-transitions), testing the bug.

Only local images are allowed.
Only local images are allowed.

gauravvvv’s picture

I have added a screen recording for reference, please see

swatidhurandhar’s picture

Status: Active » Needs review
StatusFileSize
new511.42 KB
new522.83 KB
new1.38 KB

I was able to reproduce this issue but my button size as well as color was different in first transition. Based on that, I have created a patch for Drupal 10 to fix it.

swatidhurandhar’s picture

Version: 11.x-dev » 10.1.x-dev

Changed the core version since reporter has mentioned issue with Drupal 10.

smustgrave’s picture

Version: 10.1.x-dev » 11.x-dev
Status: Needs review » Needs work
Issue tags: +Needs steps to reproduce

11.x is the current development branch.

Is this reproducible with any feature in core? Tagging for steps.

nilesh.k’s picture

Hi

This module doesn't support D11. I even tried it with the beta version, but it still gives an error. So, as of now, this module is only for D10.

manojkumar_97 made their first commit to this issue’s fork.

ameymudras made their first commit to this issue’s fork.

ameymudras’s picture

Status: Needs work » Needs review

Was able to reproduce the issue on D11 as well following the steps provided above. I have created a MR with the patch to fix

needs-review-queue-bot’s picture

Status: Needs review » Needs work

The Needs Review Queue Bot tested this issue.

While you are making the above changes, we recommend that you convert this patch to a merge request. Merge requests are preferred over patches. Be sure to hide the old patch files as well. (Converting an issue to a merge request without other contributions to the issue will not receive credit.)

shweta__sharma’s picture

StatusFileSize
new2.97 MB

I tried to replicate the issue but for me, the schedule button is looking good. Not able to replicate the issue.
Tested on Drupal 10
Thanks

shweta__sharma’s picture

Status: Needs work » Needs review
smustgrave’s picture

Status: Needs review » Postponed (maintainer needs more info)

Should he put in this status as there’s nothing to review. If someone can verify the issue, using just core. If not this will be closed

quietone’s picture

Status: Postponed (maintainer needs more info) » Closed (works as designed)
Issue tags: +Bug Smash Initiative

The issue summary says that a contrib module needs to be installed? Is this reproducible from core only?

In any case, this issue has been waiting for 10 months for someone to verify that the problem still exists after testing in #16 was not able to reproduce the problem. Therefore, it is time to close this.