Reviewed & tested by the community
Project:
Bulma CSS
Version:
3.0.x-dev
Component:
Code
Priority:
Normal
Category:
Bug report
Assigned:
Unassigned
Issue tags:
Reporter:
Created:
15 Sep 2023 at 08:08 UTC
Updated:
19 Jul 2024 at 04:21 UTC
Jump to comment: Most recent, Most recent file

Comments
Comment #2
shyam_bhattAdding a patch to resolve this issue. Please check and verify. Moving to + Needs review.
After patch:

Comment #3
shyam_bhattComment #4
Sonal Gyanani commentedApplied patch#2 it is working fine but the asterisk mark looks stretched we can fix it by providing the same height and width
Thanks
Comment #5
nilesh.k commentedHi
On my local system, patch number #2 is not applying. However, after adding the code and manually incorporating the code changes from comment #4, the asterisk mark is displaying correctly.
Comment #6
sdhruvi5142 commentedTested - https://www.drupal.org/files/issues/2023-09-15/3387496-2.patch
Testing Comments:
After applying the patch this issue is fixed. Tested the mentioned fields as well as added some more other fields to check (*) sign for the fields which are set as required and it is working for those fields also.
Status - PASS
Comment #7
shyam_bhattHi @sdhruvi5142
If testing is passed, please change status from Needs Review to RTBC.
Thanks
Comment #8
sdhruvi5142 commentedComment #9
mlncn commentedWe don't need and shouldn't use an SVG here— core and other themes simply use an asterisk. Here is an example of how Gin theme handles it:
Comment #11
preeti.chawla commentedI have created new patch with proposed solution to resolve the issue. Please review
Comment #12
preeti.chawla commentedComment #15
adarshv commentedIssue fixed. Please review MR !13 above,
Comment #16
tanuja bohra commentedVerified MR13 on 10.2.x, the asterisk is added and looks fine. RTBC++
Comment #17
sdhruvi5142 commentedHi
Verified MR13! and the fixes are working as expected.
Testing Steps followed:
1. Install Bulma theme with Drupal 10.
2. Install and enable the Web from module.
3. Create a form with some required fields. Changes observed here.
Testing result:
The asterik(*) are being properly working as expected for the required fields.
Status : PASS
Attaching SS for reference.
Comment #18
sdhruvi5142 commentedComment #19
mlncn commentedThis is fixed more generically and not only for webform in #3420212: Add asterisk to webform required fields. Can folks here confirm, testing either supported dev release (1.x or 3.x)?