Checkboxes broken in Slick/Splide/Blazy

Created on 18 December 2023, 11 months ago
Updated 3 July 2024, 5 months ago

Problem/Motivation

Unable to click on checkbox (label works).

Steps to reproduce

Configure Splide/Slick/Blazy options in those modules.

Proposed resolution

Review code for rendering checkboxes in Gin.

Remaining tasks

Enhance code.

User interface changes

Allow checkboxes to be clickable.

API changes

None.

Data model changes

None.

๐Ÿ› Bug report
Status

Closed: outdated

Version

3.0

Component

Code

Created by

๐Ÿ‡บ๐Ÿ‡ธUnited States mortona2k Seattle

Live updates comments and jobs are added and updated live.
Sign in to follow issues

Merge Requests

Comments & Activities

  • Issue created by @mortona2k
  • ๐Ÿ‡ฎ๐Ÿ‡ณIndia ravi kant Jaipur

    ravi kant โ†’ made their first commit to this issueโ€™s fork.

  • Merge request !347Checkboxes broken in Slick/Splide/Blazy โ†’ (Open) created by ravi kant
  • Pipeline finished with Success
    11 months ago
    #65862
  • Status changed to Needs review 11 months ago
  • ๐Ÿ‡ฎ๐Ÿ‡ณIndia ravi kant Jaipur

    @mortona2k
    I have fixed the issue for Splide and created MR. Also i am sure that issue will be fixing for Slick and Blazy modules due to the issue was in Gim theme and changes will effect for all modules/themes.

    Also, If you want quick fix then i can create patch for you.

  • ๐Ÿ‡ฎ๐Ÿ‡ณIndia Nitin shrivastava

    @ravi kant Thanks for the Merge Request
    MR#3 applied successfully, and the issue has been resolved.
    Now the checkbox and toogle are at the same place because of Ui is not looking breaked.

    Before

    After

  • Status changed to RTBC 10 months ago
  • Status changed to Needs work 10 months ago
  • ๐Ÿ‡จ๐Ÿ‡ญSwitzerland saschaeggi Zurich

    Left a code suggestion/question

  • ๐Ÿ‡ฎ๐Ÿ‡ณIndia Nitin shrivastava

    @saschaeggi
    I checked with the left:0, but it is not working; the issue still exists.

    Screenshot.

  • ๐Ÿ‡ฎ๐Ÿ‡ณIndia Nitin shrivastava

    @@saschaeggi
    Also i checked with this code " left: calc(50% - 45px); ".
    It is not working well on different screen sizes.

  • First commit to issue fork.
  • Status changed to Needs review 10 months ago
  • Pipeline finished with Failed
    10 months ago
    #80648
  • Pipeline finished with Failed
    10 months ago
    #80650
  • @Nitin shrivastava and @viren18febS can you please provide detailed testing steps for this issue?

  • Status changed to Closed: outdated 5 months ago
  • ๐Ÿ‡จ๐Ÿ‡ญSwitzerland saschaeggi Zurich

    I close this issue as outdated, as we completely overhauled the toggle integration.

  • Pipeline finished with Failed
    about 2 months ago
    Total: 591s
    #300821
  • Pipeline finished with Failed
    about 2 months ago
    Total: 655s
    #300840
  • Pipeline finished with Failed
    about 2 months ago
    Total: 433s
    #300850
  • Pipeline finished with Failed
    about 2 months ago
    Total: 621s
    #300856
  • Pipeline finished with Failed
    about 2 months ago
    Total: 479s
    #300913
  • Pipeline finished with Failed
    about 2 months ago
    Total: 486s
    #300968
  • Pipeline finished with Failed
    about 2 months ago
    Total: 423s
    #300971
  • Pipeline finished with Failed
    about 2 months ago
    Total: 432s
    #300989
  • Pipeline finished with Failed
    about 2 months ago
    Total: 1143s
    #301002
  • Pipeline finished with Failed
    about 2 months ago
    Total: 437s
    #301017
  • Pipeline finished with Success
    about 2 months ago
    Total: 441s
    #301033
  • Pipeline finished with Failed
    about 1 month ago
    Total: 789s
    #304209
  • Pipeline finished with Failed
    about 1 month ago
    Total: 972s
    #304234
Production build 0.71.5 2024