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

The setting of the estimated time for time tracking in the issue should also have the same UI as adding time. #33318

Open
2 tasks
seepine opened this issue Jan 18, 2025 · 2 comments
Labels
type/proposal The new feature has not been accepted yet but needs to be discussed first.

Comments

@seepine
Copy link

seepine commented Jan 18, 2025

Feature Description

# 1

  • the setting of the estimated time for time tracking in the issue should also have the same UI as adding time.
    Image

# 2

  • It is suggested to place the start timer on the outer layer instead of in the pop-up window, that is, restore to the previous mode. Because this is a relatively high-frequency operation, the actual experience in these few days shows that placing it in the pop-up window will increase the complexity.

Image

Image

Screenshots

No response

@seepine seepine added the type/proposal The new feature has not been accepted yet but needs to be discussed first. label Jan 18, 2025
@wxiaoguang
Copy link
Contributor

These changes are all from Issue time estimate, meaningful time tracking #23113

  • For the "input": I agree we should only use one style, just decide which one to keep
  • For the "Start timer" button: it was said that the UI was too complex (too many buttons), so I moved it into the dropdown. Feel free to move it out from the dropdown.

@stuzer05
Copy link
Contributor

These changes are all from Issue time estimate, meaningful time tracking #23113

* For the "input": I agree we should only use one style, just decide which one to keep

* For the "Start timer" button: it was said that the UI was too complex (too many buttons), so I moved it into the dropdown. Feel free to move it out from the dropdown.

Also, maybe default value for input should be "h" (implicit), because this is default metric used

@go-gitea go-gitea deleted a comment from print520 Jan 19, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type/proposal The new feature has not been accepted yet but needs to be discussed first.
Projects
None yet
Development

No branches or pull requests

3 participants