Skip to content

Conversation

@KooShnoo
Copy link

@KooShnoo KooShnoo commented Jan 7, 2026

Pull Request Checklist

Issue

https://otwarchive.atlassian.net/browse/AO3-6869

Purpose

Fix a layout issue in the abuse report form.

Certain browsers, seemingly a certain version of WebKit browsers, would insert extra space between some of the form inputs and their description footnotes.

I was able to reproduce on desktop Safari Version 17.6 (19618.3.11.11.5) but not on iOS.

As mentioned in the ticket,

It's related to the invisible span that's there to hold the error message. For some reason, that's started taking up space now.

Adding an empty div seems to mitigate this.

Before: 6869_before
After: 6869_after

Testing Instructions

Go to https://archiveofourown.org/abuse_reports/new or use the link in the footer.
Verify that there is no wide gap between the footnote "Please ensure this link..." and its associated text box. (link field)
Verify that there is no wide gap between the footnote "Please specify why you are contacting..." and its associated text box. (summary field)

Credit

KooShnoo he/him

@marcus8448
Copy link
Contributor

Hi KooShnoo!

Thank you so much for this pull request. Someone will be along to review it soon.

I've updated the Jira issue status to In Review so no one will mistakenly create a duplicate pull request. If you'd like the ability to comment on, assign, and transition issues in the future, you're welcome to create a Jira account! It makes things a bit easier for us on the organizational side if the Full Name on your Jira account either closely matches the name you'd like us to credit in the release notes or includes it in parentheses, e.g. "Nickname (CREDIT NAME)."

Once you've done that (or if you've already done it -- Jira has been unreliable about showing us new accounts in the admin panel lately), you can either reply here or send an email to [email protected] with both your account name and email address and we'll set up the permissions for you.

Thanks again for contributing! If you have any questions, you can contact us at the same email address listed above.

@KooShnoo
Copy link
Author

KooShnoo commented Jan 7, 2026

Oh cool, I set my name in Atlassian to KooShnoo. My email is [email protected]. Thanks!

Copy link
Collaborator

@sarken sarken left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hi, KooShnoo! Your Jira account permissions should be all set next time you log in.

Thanks for taking a look at this issue. The fix does work, but I'm not sure it's the right one for the job. It would mean adding extraneous HTML elements to every page where this issue is present (and then hopefully removing them one day when WebKit fixes the issue).

It would be much better if we could confine the fix to one file by using CSS instead.

Would something like this work?

form#new_abuse_report .footnote {
  float: none;
}

If it did, I think sandbox.css would be an okay home for it, and whenever we found another form with the issue (e.g., the account creation page), we could just add another selector to the sandbox stylesheet.

@KooShnoo
Copy link
Author

KooShnoo commented Jan 8, 2026

Ah, I see. Yes, that fix works, thanks! I hope it's okay I added some extra code to mimic the functionality of the problematic float property:

  margin-right: 0;
  width: fit-content;

I can revert it if it's no good.

Without extra code:

With extra code:

Copy link
Collaborator

@sarken sarken left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

That extra CSS looks like a good adjustment to me -- thank you!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants