Problem/Motivation

Slider arrow button is overriding the text I am adding a screen recording for better understanding.

Steps to reproduce

Install a theme
Add some content like a menu, articles etc
Try to click on the sider arrow to change the slide you will see that the slider arrow button is overriding the text

Proposed resolution

Need to fix the slider arrow button so that it won't override the text for which we need to change the placement for the slider arrow button.

Remaining tasks

None

User interface changes

Yes

API changes

None

Data model changes

None

CommentFileSizeAuthor
#7 capture dreams.mp40 bytesnupur badola
#6 After.mov912.9 KBriddhi.addweb
#4 Before.mov3.02 MBriddhi.addweb
#4 Before-Responsive.mov10.78 MBriddhi.addweb
#4 After.mov15.83 MBriddhi.addweb
#3 postpatch.mov32.33 MBhamid.ali
#3 3450600-3.patch14.01 KBhamid.ali
Screen Recording 2024-05-29 at 4.38.42 PM.mov47.25 MBhamid.ali

Issue fork capture_dreams-3450600

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

Hamid.ali created an issue. See original summary.

hamid.ali’s picture

hamid.ali commented

Will update the patch in sometime

hamid.ali’s picture

hamid.ali commented
Assigned: hamid.ali » Unassigned
Status: Active » Needs review
StatusFileSize
new 3450600-3.patch14.01 KB
new postpatch.mov32.33 MB

Added the patch for the issue also post patch screen recording.

riddhi.addweb’s picture

riddhi.addweb commented
StatusFileSize
new After.mov15.83 MB
new Before-Responsive.mov10.78 MB
new Before.mov3.02 MB

- I have tried the #3 3450600-3 Patch but it is still overriding the text in the Mobile View. I have tested this issue in "Simply test me sandbox", The issue is resolved in the desktop view, but in the mobile view, the arrow still overrides the "Virasat Solution" text. I am attaching the screenshots for the same.

hamid.ali’s picture

hamid.ali commented

@riddhi.addweb try standard viewport sizes starting at 375px for mobile.

riddhi.addweb’s picture

riddhi.addweb commented
StatusFileSize
new After.mov912.9 KB

I tried my standard viewport size, i.e. 375px, and it works fine for us. Attaching Screenshots for the same.

nupur badola’s picture

nupur badola commented
Status: Needs review » Reviewed & tested by the community
StatusFileSize
new capture dreams.mp40 bytes

Reviewed the patch 3450600-3.patch and this is working fine for me as well.

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

sarwan_verma opened merge request !7

yogesh.k made their first commit to this issue’s fork.

yogesh.k’s picture

yogesh.k commented
Version: 1.0.1 » 1.0.2
Status: Reviewed & tested by the community » Fixed
yogesh.k’s picture

yogesh.k commented
Status: Fixed » Closed (fixed)