Before fixing issue

before

CommentFileSizeAuthor
#9 Screenshot from 2024-08-07 16-14-19.png30.26 KBriddhi.addweb
#9 Screenshot from 2024-08-07 16-14-06.png38.09 KBriddhi.addweb
#8 spacing-issue-resolved.png27.72 KBjaydeep_patel
#4 BeforeMR-40.png35.34 KBkanchan bhogade
#4 AfterMR40.png36.44 KBkanchan bhogade
#3 after.png127.48 KBalok_singh
before.png158.28 KBalok_singh

Issue fork marvelous-3465147

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:

About issue forks

Comments

alok_singh created an issue. See original summary.

alok_singh opened merge request !40

alok_singh’s picture

alok_singh commented
Status: Active » Needs review
StatusFileSize
new after.png127.48 KB

After Fixing isse

after

kanchan bhogade’s picture

kanchan bhogade commented
StatusFileSize
new AfterMR40.png36.44 KB
new BeforeMR-40.png35.34 KB

Hi
I have tested MR 40 on Drupal 10
MR is applied successfully...

Test Result:
The Spacing is added but the "Add New Comments" and "Subject" spacing does not look good, and the Subject Placeholder should be on the next line.

Adding SS for reference

kanchan bhogade’s picture

kanchan bhogade commented
Status: Needs review » Needs work
hetal.solanki’s picture

hetal.solanki
Primary language English
commented

@Kanchan Bhogade

i am not able to see this form. Could you please suggest a URL? so I can fix this issue?

Thank you!!

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

jaydeep_patel’s picture

jaydeep_patel commented
Status: Needs work » Needs review
StatusFileSize
new spacing-issue-resolved.png27.72 KB

Spacing issue has been fixed and Subject field move to next line full width.

riddhi.addweb’s picture

riddhi.addweb commented
Status: Needs review » Reviewed & tested by the community
StatusFileSize
new Screenshot from 2024-08-07 16-14-06.png38.09 KB
new Screenshot from 2024-08-07 16-14-19.png30.26 KB

The mentioned page issue is resolved, & I have also checked and it is working as expected. I am attaching the Screenshots & doing RTBC for the same.

rushikesh raval made their first commit to this issue’s fork.

rushikesh raval’s picture

rushikesh raval commented
rushikesh raval’s picture

rushikesh raval commented
Status: Reviewed & tested by the community » Fixed
rushikesh raval’s picture

rushikesh raval commented
hetal.solanki’s picture

hetal.solanki
Primary language English
commented

Status: Fixed » Closed (fixed)

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