Problem/Motivation

The following two lines of text are reversed for their respective configuration fields:
- This is used for the Previous Page button on the page before this page break.
- This is used for the Next Page button on the page after this page break.

Steps to reproduce

- Build a webform
- Click the add page button
- See the description text in the pop-up

Proposed resolution

- Fix the typo

Remaining tasks

- Create a patch

User interface changes

- Change the text

API changes

- None

Data model changes

- None

CommentFileSizeAuthor
#5 After patch Next button.png120.93 KBriddhi.addweb
#5 After patch previous button.png126.53 KBriddhi.addweb
#5 Before patch Next button.png119.74 KBriddhi.addweb
#5 Before patch previous button.png123.53 KBriddhi.addweb
#2 3452551-2.patch1.37 KBandrew.wang
Screenshot 2024-05-29 at 1.30.56 PM.png328.59 KBandrew.wang

Comments

andrew.wang created an issue. See original summary.

andrew.wang’s picture

andrew.wang commented
StatusFileSize
new 3452551-2.patch1.37 KB
andrew.wang’s picture

andrew.wang commented
Status: Active » Needs review
andrew.wang’s picture

andrew.wang commented
Issue summary: View changes
riddhi.addweb’s picture

riddhi.addweb commented
StatusFileSize
new Before patch previous button.png123.53 KB
new Before patch Next button.png119.74 KB
new After patch previous button.png126.53 KB
new After patch Next button.png120.93 KB

@andrew.wang I have applied the patch successfully.
It has Resolved the issue.
Please find the attached ScreenShots for the Reference.

riddhi.addweb’s picture

riddhi.addweb commented
Status: Needs review » Reviewed & tested by the community
riddhi.addweb’s picture

riddhi.addweb commented
Status: Reviewed & tested by the community » Needs review

Currently moving this to "Needs review" for code verification.

cilefen’s picture

cilefen commented
Status: Needs review » Reviewed & tested by the community

It seems good to me too. I don't think much code review is needed here.

  • 3a3d27c7 committed on 6.2.x 
    Issue #3452551: Description texts for Previous/Next Page button are...
jrockowitz’s picture

jrockowitz commented
Status: Reviewed & tested by the community » Fixed

  • 3a3d27c7 committed on 6.x 
    Issue #3452551: Description texts for Previous/Next Page button are...

Status: Fixed » Closed (fixed)

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