Skip to content

Refresh codemirror on show pull comment tab#11100

Merged
guillep2k merged 8 commits into
go-gitea:masterfrom
zeripath:fix-10975-render-template
Apr 18, 2020
Merged

Refresh codemirror on show pull comment tab#11100
guillep2k merged 8 commits into
go-gitea:masterfrom
zeripath:fix-10975-render-template

Conversation

@zeripath
Copy link
Copy Markdown
Contributor

When trying to create a new PR the comment field does not show the default data until the field is clicked. This PR calls autoSimpleMDE.codemirror.refresh(); to cause the default data to be shown.

Fix #10975

Signed-off-by: Andrew Thornton art27@cantab.net

Fix go-gitea#10975

Signed-off-by: Andrew Thornton <art27@cantab.net>
@zeripath zeripath added the topic/ui Change the appearance of the Gitea UI label Apr 16, 2020
@zeripath zeripath added this to the 1.12.0 milestone Apr 16, 2020
@jolheiser
Copy link
Copy Markdown
Member

Does this also apply to new issues?

@GiteaBot GiteaBot added the lgtm/need 2 This PR needs two approvals by maintainers to be considered for merging. label Apr 16, 2020
@zeripath
Copy link
Copy Markdown
Contributor Author

I don't think that issues were ever affected, certainly http://localhost/gitea/administrator/merge-test/issues/new?body=rabbits+feet just works on master.

Copy link
Copy Markdown
Member

@jolheiser jolheiser left a comment

Choose a reason for hiding this comment

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

🔥

@GiteaBot GiteaBot added lgtm/need 1 This PR needs approval from one additional maintainer to be merged. and removed lgtm/need 2 This PR needs two approvals by maintainers to be considered for merging. labels Apr 16, 2020
@GiteaBot GiteaBot added lgtm/done This PR has enough approvals to get merged. There are no important open reservations anymore. and removed lgtm/need 1 This PR needs approval from one additional maintainer to be merged. labels Apr 16, 2020
@6543
Copy link
Copy Markdown
Member

6543 commented Apr 16, 2020

didnt had time to check but nice 🎉

@silverwind
Copy link
Copy Markdown
Member

silverwind commented Apr 17, 2020

I guess a proper fix would be to initialize Codemirror using the value constructor option, if that's even exposed through SimpleMDE. We should get rid of this mess at some point (#10729).

@guillep2k
Copy link
Copy Markdown
Member

Ping lg-tm

@guillep2k guillep2k merged commit 731bdef into go-gitea:master Apr 18, 2020
@zeripath
Copy link
Copy Markdown
Contributor Author

@silverwind it's already initialised with the correct value. The issue appears to be that the field is hidden and then shown.

@zeripath zeripath deleted the fix-10975-render-template branch April 18, 2020 16:12
zeripath added a commit to zeripath/gitea that referenced this pull request Apr 18, 2020
Fix go-gitea#10975

Signed-off-by: Andrew Thornton <art27@cantab.net>

Co-authored-by: John Olheiser <john.olheiser@gmail.com>
Co-authored-by: techknowlogick <techknowlogick@gitea.io>
Co-authored-by: guillep2k <18600385+guillep2k@users.noreply.github.com>
guillep2k added a commit that referenced this pull request Apr 18, 2020
Fix #10975

Signed-off-by: Andrew Thornton <art27@cantab.net>

Co-authored-by: John Olheiser <john.olheiser@gmail.com>
Co-authored-by: techknowlogick <techknowlogick@gitea.io>
Co-authored-by: guillep2k <18600385+guillep2k@users.noreply.github.com>

Co-authored-by: John Olheiser <john.olheiser@gmail.com>
Co-authored-by: techknowlogick <techknowlogick@gitea.io>
Co-authored-by: guillep2k <18600385+guillep2k@users.noreply.github.com>
@zeripath zeripath added the backport/done All backports for this PR have been created label Apr 18, 2020
ydelafollye pushed a commit to ydelafollye/gitea that referenced this pull request Jul 31, 2020
Fix go-gitea#10975

Signed-off-by: Andrew Thornton <art27@cantab.net>

Co-authored-by: John Olheiser <john.olheiser@gmail.com>
Co-authored-by: techknowlogick <techknowlogick@gitea.io>
Co-authored-by: guillep2k <18600385+guillep2k@users.noreply.github.com>
@go-gitea go-gitea locked and limited conversation to collaborators Nov 24, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

backport/done All backports for this PR have been created lgtm/done This PR has enough approvals to get merged. There are no important open reservations anymore. topic/ui Change the appearance of the Gitea UI

Projects

None yet

Development

Successfully merging this pull request may close these issues.

PR / Issue templates do not get rendered during PR creation

9 participants