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

Add option to remove "Website" form field #916

Open
flux77 opened this issue Jul 17, 2022 · 4 comments
Open

Add option to remove "Website" form field #916

flux77 opened this issue Jul 17, 2022 · 4 comments
Labels
client (Javascript) client code and CSS feature needs-contributor Someone needs to implement this. Help wanted! needs-decision Architectural/Behavioral decision by maintainers needed
Milestone

Comments

@flux77
Copy link

flux77 commented Jul 17, 2022

In the default Isso comment form, there are four form fields: content, name, email, and website. If I have no use for the "Website" form field, how do I remove it? Of course, I could remove it by using some CSS rules, but is there a better solution?

Perhaps there could be a data-isso-website-field option that controls whether or not to render the "Website" field.

Isso

@ix5 ix5 added needs-contributor Someone needs to implement this. Help wanted! client (Javascript) client code and CSS feature needs-decision Architectural/Behavioral decision by maintainers needed labels Jul 18, 2022
@ix5 ix5 added this to the 0.13 milestone Jul 18, 2022
@ix5
Copy link
Member

ix5 commented Jul 19, 2022

This could probably be done with CSS, but in the interest of accessibility, we should have a server-side options matrix:

Field Status
Name (Author) Disabled / Optional / Required
Email Disabled / Optional / Required
Website Disabled / Optional / Required

@ix5 ix5 modified the milestones: 0.13, 0.14 Jan 22, 2024
@ix5 ix5 modified the milestones: 0.14, 0.13.1 May 9, 2024
@AndreyPilipenko
Copy link

any updates on it?

@CaffeineFueled1
Copy link

Since the option is not yet available, hiding it with CSS has worked for me:

/* ISSO COMMENTS */
label[for=isso-postbox-website] {
  display: none !important;
}

input#isso-postbox-website {
    display: none;

That hides the website field. Replace website with email to hide the email field.

@jelmer
Copy link
Member

jelmer commented Jul 8, 2024

I wonder if we should document that CSS rather than adding another option.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
client (Javascript) client code and CSS feature needs-contributor Someone needs to implement this. Help wanted! needs-decision Architectural/Behavioral decision by maintainers needed
Projects
None yet
Development

No branches or pull requests

5 participants