[PR #5239] [CLOSED] Fix comment box not showing on PR commit replies #17717

Closed
opened 2025-11-02 15:15:26 -06:00 by GiteaMirror · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/go-gitea/gitea/pull/5239
Author: @jamesa
Created: 10/31/2018
Status: Closed

Base: masterHead: fix-comment-box


📝 Commits (3)

  • d9abe4e Fix comment box not showing on PR commit replies
  • 41f86ad Merge branch 'master' into fix-comment-box
  • 137116b Merge branch 'master' into fix-comment-box

📊 Changes

1 file changed (+2 additions, -1 deletions)

View changed files

📝 public/js/index.js (+2 -1)

📄 Description

Fixes #5237

The form that gets revealed when clicking on the "Reply" button mentioned in #5237, and the button, do not share the same parent. The button ends up getting wrapped in a link which causes this issue.

I have no idea how or why this worked on the English translation though. Didn't dig in that deep. :)


🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.

## 📋 Pull Request Information **Original PR:** https://github.com/go-gitea/gitea/pull/5239 **Author:** [@jamesa](https://github.com/jamesa) **Created:** 10/31/2018 **Status:** ❌ Closed **Base:** `master` ← **Head:** `fix-comment-box` --- ### 📝 Commits (3) - [`d9abe4e`](https://github.com/go-gitea/gitea/commit/d9abe4ea3098cc693616e26c358967c63498b119) Fix comment box not showing on PR commit replies - [`41f86ad`](https://github.com/go-gitea/gitea/commit/41f86ad04d550d62c7d79faf846a9517d94ab7b2) Merge branch 'master' into fix-comment-box - [`137116b`](https://github.com/go-gitea/gitea/commit/137116b40befb8e2519431a73a081c09c133a2c9) Merge branch 'master' into fix-comment-box ### 📊 Changes **1 file changed** (+2 additions, -1 deletions) <details> <summary>View changed files</summary> 📝 `public/js/index.js` (+2 -1) </details> ### 📄 Description Fixes #5237 The form that gets revealed when clicking on the "Reply" button mentioned in #5237, and the button, do not share the same parent. The button ends up getting wrapped in a link which causes this issue. I have no idea how or why this worked on the English translation though. Didn't dig in that deep. :) --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>
GiteaMirror added the pull-request label 2025-11-02 15:15:26 -06:00
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: github-starred/gitea#17717