Upload form fields need changes #94

Open
opened 2025-06-22 19:58:13 +00:00 by davmo · 2 comments
Owner

The upload form is presented if the browser does not permit JavaScript.

The part of the form requesting details of the show being uploaded has some layout problems:

  • The "Format" field (which asks for the format of the notes provided later in the form) is placed first in the section. It refers to "show notes above" since it used to follow the "Notes" field.
  • The "Tags" field is difficult to see. The label is centred in relation to the tag entry widget and the text which follows, and the entry widget is difficult to distinguish as a consequence.
  • Ideally all field labels should be aligned to the top of each field to aid readability, but this is probably a matter of opinion.

A screenshot of the relevant part of the form is attached.

The upload form is presented if the browser does not permit JavaScript. The part of the form requesting details of the show being uploaded has some layout problems: - The "Format" field (which asks for the format of the notes provided later in the form) is placed first in the section. It refers to "show notes above" since it used to follow the "Notes" field. - The "Tags" field is difficult to see. The label is centred in relation to the tag entry widget and the text which follows, and the entry widget is difficult to distinguish as a consequence. - Ideally all field labels should be aligned to the top of each field to aid readability, but this is probably a matter of opinion. A screenshot of the relevant part of the form is attached.
rho_n self-assigned this 2025-06-26 02:40:54 +00:00
Owner

@davmo @ken_fallon I wasn't able to test this running as PHP. I made a copy of the upload.php and renamed it upload.html. I added the <head>...</head> lines from header.html so that I could test the changes to CSS I made to help with the layout problems. I think it resolves Dave's issues. I did test it without javascript and didn't see the issue with the Format field being in the wrong place. I don't know if not running it as PHP would affect the layout and hide the issue.

@davmo @ken_fallon I wasn't able to test this running as PHP. I made a copy of the upload.php and renamed it upload.html. I added the <head>...</head> lines from header.html so that I could test the changes to CSS I made to help with the layout problems. I think it resolves Dave's issues. I did test it without javascript and didn't see the issue with the Format field being in the wrong place. I don't know if not running it as PHP would affect the layout and hide the issue.
Owner

Here is the PR with the changes I made: #95

Here is the PR with the changes I made: https://repo.anhonesthost.net/HPR/hpr_hub/pulls/95
Sign in to join this conversation.
No Label
2 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: HPR/hpr_hub#94
No description provided.