Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Tiranti radio buttons are rendered incorrectly #460

Open
noahgibbs opened this issue Nov 17, 2023 · 4 comments
Open

Tiranti radio buttons are rendered incorrectly #460

noahgibbs opened this issue Nov 17, 2023 · 4 comments
Labels
bug Something isn't working good first issue Good for newcomers KWoC-2023 IIT Kharagpur Winter of Code 2023 - good for new contributors

Comments

@noahgibbs
Copy link
Collaborator

noahgibbs commented Nov 17, 2023

For instance if you run:

SCARPE_HTML_RENDERER=tiranti ./exe/scarpe --dev examples/radio/radio.rb

This is what I see:

Screenshot 2023-11-17 at 13 32 41

The buttons aren't correctly keeping the text from overlapping them. This is probably a matter of generating correct Bootstrap markup. It should be separate from the problem in issue #383 -- this one is (I think) just a markup issue. But the radio buttons won't group correctly until that issue is fixed anyway.

I normally check the Bootswatch theme sample sites for markup (https://bootswatch.com/sketchy/). Looks like for radio buttons we want something like their example:

  <div class="form-check">
  <input class="form-check-input" type="radio" name="optionsRadios" id="optionsRadios1" value="option1" checked>
  <label class="form-check-label" for="optionsRadios1">
  Option one is this and that&mdash;be sure to include why it's great
  </label>
  </div>
  <div class="form-check">
  <input class="form-check-input" type="radio" name="optionsRadios" id="optionsRadios2" value="option2">
  <label class="form-check-label" for="optionsRadios2">
  Option two can be something else and selecting it will deselect option one
  </label>
  </div>
  <div class="form-check disabled">
  <input class="form-check-input" type="radio" name="optionsRadios" id="optionsRadios3" value="option3" disabled>
  <label class="form-check-label" for="optionsRadios3">
  Option three is disabled
  </label>
  </div>
@noahgibbs noahgibbs added bug Something isn't working good first issue Good for newcomers KWoC-2023 IIT Kharagpur Winter of Code 2023 - good for new contributors labels Nov 17, 2023
@HUDASULTANA
Copy link

hey @noahgibbs Do you mean to say that we need to increase the space between buttons and text?

@noahgibbs
Copy link
Collaborator Author

For me, the buttons literally cover the whole first word. So yes, but most importantly, by default the buttons should not be on top of the text. It's probably some kind of a simple CSS or Bootstrap issue.

@HUDASULTANA
Copy link

HUDASULTANA commented Jan 6, 2024 via email

@noahgibbs
Copy link
Collaborator Author

Something that may help... there are demo pages for Bootswatch themes, including Sketchy, the one we use by default with Tiranti. https://bootswatch.com/sketchy/

You can search for "radio button" on that page. They're using the same CSS classes and styles there, and they're working. So presumably part of our problem is that we're not doing the right thing from that page.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working good first issue Good for newcomers KWoC-2023 IIT Kharagpur Winter of Code 2023 - good for new contributors
Projects
None yet
Development

No branches or pull requests

2 participants