-
-
Notifications
You must be signed in to change notification settings - Fork 5.5k
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
Track issue changes, join labels #834
Comments
The initial feature PR is #788 |
Could this be done in the UI? having something that reads the event-lines and checks username + "added"/"removed" + time_since and concatenating them? |
If we changed more than two labels, how to display on the UI? |
$(".event").each(function(index) {
curr = $(this);
p = curr.prev();
if (p != undefined && p.hasClass("event")) {
event = "added"
if ($("span.text", p).text().includes("removed")) {
event = "removed"
}
if ($("span > .time-since", p).text() == $("span > .time-since", curr).text() &&
$("span.text", curr).text().includes(event)) {
$("span > .label", curr).after($("span > .label", p));
p.remove();
}
}
}); Edit: The above code is ugly AF and only serves as a proof-of-concept. If someone makes a PR for this, clean up the code first 😂 |
Hint: If the |
This looks good |
We could also turn them into Vue-components now that we use Vue in another place |
@mbger currently nobody working on this. |
This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs during the next 2 weeks. Thank you for your contributions. |
Unstale 👍 |
this was recently merged, can't find PR for it though right now, but you can see that on try.gitea.io it works. |
When changing/adding/removing multiple labels at the same time, each "event" is showing on their own seperate line, as show below
Wouldn't it be more cleaner to join all the events, so they're represented in one line instead, like this:
Want to back this issue? Post a bounty on it! We accept bounties via Bountysource.
The text was updated successfully, but these errors were encountered: