Skip to content

Issues: Lay out the new issue page like GitHub's, match list and state details - #78

Merged
whereareiam merged 1 commit into
devfrom
fix/issue-pages
Oct 4, 2026
Merged

whereareiam merged 1 commit into
devfrom
fix/issue-pages

Conversation

@whereareiam

Copy link
Copy Markdown
Owner

Stacked on #77 (base is its branch).

Comparison

Measured against GitHub in a signed-in browser on a temporary private repository (open and closed issues, labels, a comment).

GitHub Before After
New issue page avatar + "Create new issue" heading, title field, boxed editor with tab header, attachment line, 32px buttons stock Forgejo: repository header bar, boxed form with arrow, loose toolbar, dashed drop area, 34px button GitHub's layout
New issue title field 32px 35px 32px
New issue editor height 436px text area 132px 300px minimum
Closed issue icon in the list purple red purple
List title weight 500 600 500
Issue state label weight 600 500 600

Already matching, unchanged: list sidebar, search bar, 48px list header, 16px/24px titles, 12px meta line, 16px state icons; issue page title (32px/400), full-width comment boxes with header avatars, 912px content column, sidebar headings.

Not changed:

  • "Add a title" / "Add a description" field labels and the "Create more" checkbox: Forgejo has neither, and the labels would be hard-coded English.
  • "Closed as not planned" (grey on GitHub): Forgejo has one closed state.
  • GitHub's row meta is owner/repo#N · author opened …; Forgejo's is #N opened … by author.

Editor styles

styles/components/issues/editor.ts now emits the editor and attachment rules once, for a list of hosts (comment form, new issue page, review panel, diff comment form), instead of interpolating the same fragment in each file. Dark theme CSS is 220,133 bytes against the 225,280 limit the build test enforces.

Templates

templates/repo/issue/new.tmpl overrides Forgejo's: it drops repo/header, adds the page class and a heading from the repo.issues.new translation, and still renders Forgejo's own repo/issue/new_form.

Verification

Checked in the preview: new issue page, issue list (open and closed), an issue page, and the three other editors (comment form, review panel, diff comment form) after the consolidation. bun vitest (403 passed), bun check and Prettier pass. Screenshots are not attached.

🤖 Generated with Claude Code

…e details

Compared against GitHub signed in (issue list, issue page, closed issues,
new issue page):
- The new issue page was stock Forgejo. It now has GitHub's layout: the
  avatar and a heading, a 32px title field, the shared comment editor, a
  plain attachment line and a 32px submit button, without the repository
  header bar or the box around the form.
- Closed issues show a purple icon in the list instead of red.
- Issue and pull request titles in the lists are 500, the state label on
  an issue page is 600.

The comment editor's rules are now written once for every host instead
of being repeated per page, which also keeps the theme files under the
size limit.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@whereareiam whereareiam added the change Intentional style, template, or compatibility improvements label Oct 4, 2026
Base automatically changed from fix/pull-polish to dev October 4, 2026 10:24
@whereareiam
whereareiam merged commit 60abd97 into dev Oct 4, 2026
7 checks passed
@whereareiam
whereareiam deleted the fix/issue-pages branch October 4, 2026 10:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

change Intentional style, template, or compatibility improvements

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant