Opened 3 months ago

Last modified 11 hours ago

#5583 new enhancement

"Create Account" and "Start an Instance" are squished together

Reported by: anongoblin Owned by:
Priority: trivial Milestone:
Component: graphic design/interface Keywords: css, button
Cc: muto.lachen@… Parent Tickets:

Description

On the main starting page of Mediagoblin (when the user is logged out), there are two buttons:
"Create an account on this site" and
"Set up MediaGoblin on your own server".

The two buttons appear to have no margin between them, forcing them to be as close together as possible.

This is using the "base.css" theme.

Subtickets

Attachments (3)

issue_5583.patch (693 bytes) - added by anongoblin 3 months ago.
A patch file addressing the issue (add margin: 1px to both button classes)
Screen Shot 2020-04-02 at 02.51.31.png (106.9 KB) - added by Ben Sturmfels 6 days ago.
Screenshot without patch applied
firefox-nopatch.png (8.7 KB) - added by anongoblin 5 days ago.
A Firefox screenshot demonstrating that the buttons are close together.

Download all attachments as: .zip

Change History (10)

Changed 3 months ago by anongoblin

Attachment: issue_5583.patch added

A patch file addressing the issue (add margin: 1px to both button classes)

comment:1 Changed 6 days ago by Ben Sturmfels

Cc: muto.lachen@… added

Hi Muto Lachen,

Thanks for taking the time to contribute, and sorry for the delay in replying!

Would you mind attaching an image of the buttons that are too close together? I've attached an image of what I'm seeing, before applying your patch - it looks ok on the default theme. What theme do you have configured in your mediagoblin.ini?

Regards,
Ben

Changed 6 days ago by Ben Sturmfels

Screenshot without patch applied

Changed 5 days ago by anongoblin

Attachment: firefox-nopatch.png added

A Firefox screenshot demonstrating that the buttons are close together.

comment:2 in reply to:  1 ; Changed 5 days ago by anongoblin

Replying to Ben Sturmfels:

Would you mind attaching an image of the buttons that are too close together? I've attached an image of what I'm seeing, before applying your patch - it looks ok on the default theme. What theme do you have configured in your mediagoblin.ini?

I replied via email, but I don't know if it will be publicly visible.
On Firefox 72 and Icecat 68, the buttons are close together (attached image: firefox-nopatch.png).
On KDE's Falkon, this issue does not exist, *however*, when I apply the patch to fix the Firefox issue, strangely enough, Falkon doesn't see any additional spacing.

I didn't specify a theme in mediagoblin.ini (which should make Mediagoblin use the default theme).

Last edited 5 days ago by anongoblin (previous) (diff)

comment:3 in reply to:  2 ; Changed 5 days ago by Ben Sturmfels

Replying to anongoblin:

I replied via email, but I don't know if it will be publicly visible.

I can't see your image sorry. Could you please us the "attach file" button to upload your screenshot?

comment:4 in reply to:  3 ; Changed 5 days ago by Ben Card

Replying to Ben Sturmfels:

I can't see your image sorry. Could you please us the "attach file" button to upload your screenshot?

Hey there. It's still me (Muto), I just registered an account.
I did attach the file. Underneath the original post, the "attachments" dropdown menu should include all attachments regarding this ticket. Both our screenshots of the buttons without the patch applied are there.

In case the link is not showing up for you:
https://issues.mediagoblin.org/attachment/ticket/5583/firefox-nopatch.png

Last edited 5 days ago by Ben Card (previous) (diff)

comment:5 in reply to:  4 ; Changed 20 hours ago by Ben Sturmfels

Replying to Ben Card:

Hey there. It's still me (Muto), I just registered an account.
I did attach the file. Underneath the original post, the "attachments" dropdown menu should include all attachments regarding this ticket. Both our screenshots of the buttons without the patch applied are there.

In case the link is not showing up for you:
https://issues.mediagoblin.org/attachment/ticket/5583/firefox-nopatch.png

Thanks Ben - I can see the screenshot now. I've tried this on a range of devices and browsers and I'm not able to replicate it. I've tried Icecat 68.6.0esr (Guix), Icecat 52.3.0 (Trisquel) Chromium 80.0.3987.149 (Guix) and a couple of phones.

The fonts in your screenshot also seem a little larger than what I'm seeing. Is everything loading correctly in Firefox's developer console? Is it possible that you have a plugin blocking the include Lato font or something?

comment:6 Changed 20 hours ago by Ben Sturmfels

Also just tried Firefox 68.5.0esr (Debian) and couldn't replicate the spacing issue. This is on a new install based on the master branch of MediaGoblin.

comment:7 in reply to:  5 Changed 11 hours ago by Ben Card

Replying to Ben Sturmfels:

The fonts in your screenshot also seem a little larger than what I'm seeing. Is everything loading correctly in Firefox's developer console? Is it possible that you have a plugin blocking the include Lato font or something?

Aha! It's good that you mentioned fonts. It totally slipped my mind that I don't allow websites to load custom fonts.

This is an error on my side. It is not an issue at all. This ticket can be closed. Sorry for wasting your time!

Note: See TracTickets for help on using tickets.