Skip to content

Conversation

Sorien
Copy link
Contributor

@Sorien Sorien commented Apr 19, 2020

Fix timeline comments last section borders

image

Fix DropZone and buttons left alignment

image

Update review comment form to look the same as others

image

Fix Time Tracker Add Time button left border

image

Remove/Tidy up some classes

@codecov-io
Copy link

codecov-io commented Apr 19, 2020

Codecov Report

Merging #11139 into master will decrease coverage by 0.00%.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff             @@
##           master   #11139      +/-   ##
==========================================
- Coverage   43.84%   43.83%   -0.01%     
==========================================
  Files         607      607              
  Lines       86893    86893              
==========================================
- Hits        38098    38090       -8     
- Misses      44093    44099       +6     
- Partials     4702     4704       +2     
Impacted Files Coverage Δ
modules/indexer/stats/queue.go 62.50% <0.00%> (-18.75%) ⬇️
modules/indexer/stats/db.go 40.62% <0.00%> (-9.38%) ⬇️
modules/git/utils.go 65.67% <0.00%> (-4.48%) ⬇️
services/pull/check.go 52.43% <0.00%> (-3.05%) ⬇️
modules/git/command.go 86.95% <0.00%> (-2.61%) ⬇️
models/unit.go 41.97% <0.00%> (-2.47%) ⬇️
modules/git/repo.go 49.79% <0.00%> (-2.10%) ⬇️
models/issue.go 52.41% <0.00%> (+0.46%) ⬆️
services/pull/pull.go 33.96% <0.00%> (+0.57%) ⬆️
modules/notification/webhook/webhook.go 43.59% <0.00%> (+0.58%) ⬆️
... and 1 more

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 b7c82cd...48a4363. Read the comment docs.

@GiteaBot GiteaBot added the lgtm/need 2 This PR needs two approvals by maintainers to be considered for merging. label Apr 19, 2020
@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 20, 2020
@zeripath zeripath added the topic/ui Change the appearance of the Gitea UI label Apr 22, 2020
@zeripath zeripath added this to the 1.12.0 milestone Apr 22, 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 22, 2020
@Sorien Sorien marked this pull request as draft April 22, 2020 21:47
@lunny
Copy link
Member

lunny commented May 2, 2020

Is this still a draft?

@6543
Copy link
Member

6543 commented May 2, 2020

a draft with lgtm 😆 - @Sorien ☝️

@Sorien
Copy link
Contributor Author

Sorien commented May 2, 2020

I wanted to try if it's possible to align buttons by switching margin from right to left, I will try to look at it tomorrow.

@Sorien Sorien force-pushed the prettify-timeline-3 branch from 1933fee to 84e3ddb Compare May 3, 2020 14:09
@Sorien
Copy link
Contributor Author

Sorien commented May 3, 2020

ok should be ready, rebased commits (sorry for that), new commits from
41a427a

@Sorien Sorien marked this pull request as ready for review May 3, 2020 14:10
@Sorien
Copy link
Contributor Author

Sorien commented May 3, 2020

I have a question why there is file dropzone area in the preview tab?

Copy link
Member

@6543 6543 left a comment

Choose a reason for hiding this comment

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

I need a look again

@guillep2k guillep2k requested a review from silverwind May 3, 2020 18:00
@guillep2k guillep2k added lgtm/need 2 This PR needs two approvals by maintainers to be considered for merging. and removed lgtm/done This PR has enough approvals to get merged. There are no important open reservations anymore. labels May 3, 2020
@guillep2k
Copy link
Member

@Sorien lint fails.

@GiteaBot GiteaBot added lgtm/done This PR has enough approvals to get merged. There are no important open reservations anymore. and removed lgtm/need 2 This PR needs two approvals by maintainers to be considered for merging. labels May 3, 2020
@guillep2k
Copy link
Member

@silverwind Care to take another look?

@silverwind
Copy link
Member

I have a question why there is file dropzone area in the preview tab?

Not sure what you mean but I'd say a dropzone should be outside the write/preview tabs.

@techknowlogick techknowlogick merged commit 1982194 into go-gitea:master May 14, 2020
@techknowlogick techknowlogick added the skip-changelog This PR is irrelevant for the (next) changelog, for example bug fixes for unreleased features. label May 14, 2020
ydelafollye pushed a commit to ydelafollye/gitea that referenced this pull request Jul 31, 2020
* Fix timeline comments sections borders

* Fix files dropzone right alignment

* Update review comment form and controls

* Clear segment style from comment form tabs

* Remove segment class from pulls tab

* Fix Time Tracker - Add Time button border

* Fix buttons right alignment

* Markdown tab minimal height

* Fix DropZone hover effect

* Fix Lint + remove unused controls class

Co-authored-by: Lunny Xiao <[email protected]>
Co-authored-by: zeripath <[email protected]>
@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

lgtm/done This PR has enough approvals to get merged. There are no important open reservations anymore. skip-changelog This PR is irrelevant for the (next) changelog, for example bug fixes for unreleased features. topic/ui Change the appearance of the Gitea UI

Projects

None yet

Development

Successfully merging this pull request may close these issues.

10 participants