The Wayback Machine - https://web.archive.org/web/20220416093051/https://github.com/go-gitea/gitea/pull/17375
Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Fix SVG side by side comparison link #17375

Merged
merged 8 commits into from Oct 21, 2021
Merged

Fix SVG side by side comparison link #17375

merged 8 commits into from Oct 21, 2021

Conversation

saitho
Copy link
Contributor

@saitho saitho commented Oct 20, 2021

Fixes issue described in #17112 by making the tab names unique using the "Index" property of a file.

I also fixed a display bug that occurs when selecting the "Swipe" tab and scrolling the page. In that case, the points were above the other UI elements.

zindex-bug

saitho added 2 commits Oct 20, 2021
@saitho
Remove swipe-bar z-index
7f81876 
Fixes position of swipe-bar so it does not overlay other UI components when scrolling.

Signed-off-by: Mario Lubenka <[email protected]>
@saitho
Unique names for image tabs in pull request
8a0fef1 
Define unique names for image tabs in pull requests, in order to toggle tabs correctly when multiple are displayed on one page.

Signed-off-by: Mario Lubenka <[email protected]>
@saitho saitho changed the title Fix SVG side by side comparison Fix SVG side by side comparison link Oct 20, 2021
@techknowlogick techknowlogick added this to the 1.16.0 milestone Oct 20, 2021
@codecov-commenter
Copy link

@codecov-commenter codecov-commenter commented Oct 21, 2021

Codecov Report

Merging #17375 (6968190) into main (83df0ca) will decrease coverage by 0.01%.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff             @@
##             main   #17375      +/-   ##
==========================================
- Coverage   45.27%   45.26%   -0.02%     
==========================================
  Files         788      788              
  Lines       88430    88430              
==========================================
- Hits        40041    40028      -13     
- Misses      41917    41922       +5     
- Partials     6472     6480       +8
Impacted Files Coverage Δ
modules/notification/mail/mail.go 35.29% <0.00%> (-2.95%) ⬇️
modules/notification/ui/ui.go 60.86% <0.00%> (-1.45%) ⬇️
models/notification.go 65.28% <0.00%> (-0.88%) ⬇️
modules/queue/queue_bytefifo.go 59.28% <0.00%> (-0.60%) ⬇️
models/issue_comment.go 51.31% <0.00%> (-0.59%) ⬇️
modules/queue/workerpool.go 48.85% <0.00%> (-0.39%) ⬇️
models/unit.go 43.83% <0.00%> (+2.73%) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 83df0ca...6968190. Read the comment docs.

@wxiaoguang wxiaoguang merged commit 67561e7 into go-gitea:main Oct 21, 2021
2 checks passed
@silverwind silverwind mentioned this pull request Oct 21, 2021
Closed
@lunny
Copy link
Member

@lunny lunny commented Oct 21, 2021

Please send backport to v1.15

zeripath added a commit to zeripath/gitea that referenced this issue Oct 21, 2021
@saitho @lafriks
@lunny @wxiaoguang @zeripath
Fix SVG side by side comparison link (go-gitea#17375)
5075be9 
Backport go-gitea#17375

Define unique names for image tabs in pull requests, in order to toggle tabs correctly when multiple are displayed on one page.

Fixes position of swipe-bar so it does not overlay other UI components when scrolling.

Signed-off-by: Mario Lubenka <[email protected]>

Co-authored-by: Lauris BH <[email protected]>
Co-authored-by: Lunny Xiao <[email protected]>
Co-authored-by: wxiaoguang <[email protected]>
@zeripath zeripath mentioned this pull request Oct 21, 2021
Merged
zeripath added a commit to zeripath/gitea that referenced this issue Oct 21, 2021
@zeripath @zeripath
Changelog 1.15.5
9ae77e1 
* SECURITY
  * Upgrade Bluemonday to v1.0.16 (go-gitea#17372) (go-gitea#17374)
  * Ensure correct SSH permissions check for private and restricted users (go-gitea#17370) (go-gitea#17373)
* BUGFIXES
  * Prevent NPE in CSV diff rendering when column removed (go-gitea#17018) (go-gitea#17377)
  * Offer rsa-sha2-512 and rsa-sha2-256 algorithms in internal SSH (go-gitea#17281) (go-gitea#17376)
  * Don't panic if we fail to parse U2FRegistration data (go-gitea#17304) (go-gitea#17371)
  * Ensure popup text is aligned left (backport for 1.15) (go-gitea#17343)
  * Ensure that git daemon export ok is created for mirrors (go-gitea#17243) (go-gitea#17306)
  * Disable core.protectNTFS (go-gitea#17300) (go-gitea#17302)
  * Use pointer for wrappedConn methods (go-gitea#17295) (go-gitea#17296)
  * AutoRegistration is supposed to be working with disabled registration (backport) (go-gitea#17292)
  * Handle duplicate keys on GPG key ring (go-gitea#17242) (go-gitea#17284)
  * Fix SVG side by side comparison link (go-gitea#17375) (go-gitea#17391)

Signed-off-by: Andrew Thornton <[email protected]>
@zeripath zeripath mentioned this pull request Oct 21, 2021
Merged
zeripath added a commit that referenced this issue Oct 21, 2021
@saitho @zeripath @zeripath
Fix SVG side by side comparison link (#17375) (#17391)
cae8c63 
Backport #17375

Define unique names for image tabs in pull requests, in order to toggle tabs correctly when multiple are displayed on one page.

Fixes position of swipe-bar so it does not overlay other UI components when scrolling.

Signed-off-by: Mario Lubenka <[email protected]>

Co-authored-by: Mario Lubenka <[email protected]>
6543 pushed a commit that referenced this issue Oct 21, 2021
@zeripath @zeripath
Changelog 1.15.5 (#17392)
3aecea2 
* SECURITY
  * Upgrade Bluemonday to v1.0.16 (#17372) (#17374)
  * Ensure correct SSH permissions check for private and restricted users (#17370) (#17373)
* BUGFIXES
  * Prevent NPE in CSV diff rendering when column removed (#17018) (#17377)
  * Offer rsa-sha2-512 and rsa-sha2-256 algorithms in internal SSH (#17281) (#17376)
  * Don't panic if we fail to parse U2FRegistration data (#17304) (#17371)
  * Ensure popup text is aligned left (backport for 1.15) (#17343)
  * Ensure that git daemon export ok is created for mirrors (#17243) (#17306)
  * Disable core.protectNTFS (#17300) (#17302)
  * Use pointer for wrappedConn methods (#17295) (#17296)
  * AutoRegistration is supposed to be working with disabled registration (backport) (#17292)
  * Handle duplicate keys on GPG key ring (#17242) (#17284)
  * Fix SVG side by side comparison link (#17375) (#17391)

Signed-off-by: Andrew Thornton <[email protected]>
zeripath added a commit to zeripath/gitea that referenced this issue Oct 22, 2021
@zeripath @zeripath
Changelog 1.15.5 (go-gitea#17392)
26265b0 
Frontpront go-gitea#17392

* SECURITY
  * Upgrade Bluemonday to v1.0.16 (go-gitea#17372) (go-gitea#17374)
  * Ensure correct SSH permissions check for private and restricted users (go-gitea#17370) (go-gitea#17373)
* BUGFIXES
  * Prevent NPE in CSV diff rendering when column removed (go-gitea#17018) (go-gitea#17377)
  * Offer rsa-sha2-512 and rsa-sha2-256 algorithms in internal SSH (go-gitea#17281) (go-gitea#17376)
  * Don't panic if we fail to parse U2FRegistration data (go-gitea#17304) (go-gitea#17371)
  * Ensure popup text is aligned left (backport for 1.15) (go-gitea#17343)
  * Ensure that git daemon export ok is created for mirrors (go-gitea#17243) (go-gitea#17306)
  * Disable core.protectNTFS (go-gitea#17300) (go-gitea#17302)
  * Use pointer for wrappedConn methods (go-gitea#17295) (go-gitea#17296)
  * AutoRegistration is supposed to be working with disabled registration (backport) (go-gitea#17292)
  * Handle duplicate keys on GPG key ring (go-gitea#17242) (go-gitea#17284)
  * Fix SVG side by side comparison link (go-gitea#17375) (go-gitea#17391)

Signed-off-by: Andrew Thornton <[email protected]>
zeripath added a commit to zeripath/gitea that referenced this issue Oct 22, 2021
@zeripath @zeripath
Changelog 1.15.5 (go-gitea#17392)
e3365ce 
Frontport go-gitea#17392

* SECURITY
  * Upgrade Bluemonday to v1.0.16 (go-gitea#17372) (go-gitea#17374)
  * Ensure correct SSH permissions check for private and restricted users (go-gitea#17370) (go-gitea#17373)
* BUGFIXES
  * Prevent NPE in CSV diff rendering when column removed (go-gitea#17018) (go-gitea#17377)
  * Offer rsa-sha2-512 and rsa-sha2-256 algorithms in internal SSH (go-gitea#17281) (go-gitea#17376)
  * Don't panic if we fail to parse U2FRegistration data (go-gitea#17304) (go-gitea#17371)
  * Ensure popup text is aligned left (backport for 1.15) (go-gitea#17343)
  * Ensure that git daemon export ok is created for mirrors (go-gitea#17243) (go-gitea#17306)
  * Disable core.protectNTFS (go-gitea#17300) (go-gitea#17302)
  * Use pointer for wrappedConn methods (go-gitea#17295) (go-gitea#17296)
  * AutoRegistration is supposed to be working with disabled registration (backport) (go-gitea#17292)
  * Handle duplicate keys on GPG key ring (go-gitea#17242) (go-gitea#17284)
  * Fix SVG side by side comparison link (go-gitea#17375) (go-gitea#17391)

Signed-off-by: Andrew Thornton <[email protected]>
@zeripath zeripath mentioned this pull request Oct 22, 2021
Merged
Chianina pushed a commit to Chianina/gitea that referenced this issue Mar 28, 2022
@saitho @lafriks
@lunny @wxiaoguang Stelios Malathouras
Fix SVG side by side comparison link (go-gitea#17375)
7d8c688 
* Remove swipe-bar z-index

Fixes position of swipe-bar so it does not overlay other UI components when scrolling.

Signed-off-by: Mario Lubenka <[email protected]>

* Unique names for image tabs in pull request

Define unique names for image tabs in pull requests, in order to toggle tabs correctly when multiple are displayed on one page.

Signed-off-by: Mario Lubenka <[email protected]>

Co-authored-by: Lauris BH <[email protected]>
Co-authored-by: Lunny Xiao <[email protected]>
Co-authored-by: wxiaoguang <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@lafriks lafriks

@zeripath zeripath

@delvh delvh

Assignees
No one assigned
Labels
backport/done backport/v1.15 kind/bug kind/ui lgtm/done
Milestone
1.16.0
Development

Successfully merging this pull request may close these issues.

None yet
10 participants
@saitho @codecov-commenter @lunny @lafriks @zeripath @delvh @techknowlogick @wxiaoguang @GiteaBot @6543