Skip to content

when -> tasks go to grey, due to them being merged; they seem to jump#1045

Merged
jefking merged 2 commits into
mainfrom
moltenhub-when-tasks-go-to-grey-due-to-them-being
Jun 2, 2026
Merged

when -> tasks go to grey, due to them being merged; they seem to jump#1045
jefking merged 2 commits into
mainfrom
moltenhub-when-tasks-go-to-grey-due-to-them-being

Conversation

@moltenbot000
Copy link
Copy Markdown
Contributor

Proposed changes from Molten.Bot

This PR implements the requested changes described below.
Built using AI-assisted engineering and reviewed before submission.
Only relevant files were modified.

Original task prompt:

Update the existing pull request to address review feedback.

Original task prompt:
  when -> tasks go to grey, due to them being merged; they seem to jump around alot on the ui; and there is duplication.
  They should not be duplicated, they should just have their css style updated. same with not bouncing around...

Pull request:
- URL: https://github.com/Molten-Bot/moltenhub-code/pull/1044
- Title: when -> tasks go to grey, due to them being merged; they seem to jump
- Head branch to update: moltenhub-when-tasks-go-to-grey-due-to-them-being
- Base branch: main

Review feedback to address:
1. review by jefking (COMMENTED):
  [Medium] internal/web/static/index.html:10502 - Same-ID reorders are frozen

Requirements:
- Work on the existing pull request branch named above; do not create a separate PR.
- Keep the original task scope, but make the changes needed to satisfy the review feedback.
- Add or update tests when the feedback describes behavior that can regress.
- Preserve unrelated code and avoid broad refactors.
- Push the fix to the same PR branch so the existing pull request is updated.

Curious how this was built? See how AI agents can help with your own projects: MoltenBot Code

moltenhub-bot and others added 2 commits June 1, 2026 23:57
Co-authored-by: Molten Bot 000 <260473928+moltenbot000@users.noreply.github.com>
@moltenbot000 moltenbot000 requested a review from jefking June 2, 2026 00:06
Copy link
Copy Markdown
Collaborator

@jefking jefking left a comment

Choose a reason for hiding this comment

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

Positive

  • The PR removes the same-ID reorder freeze and keeps existing task cards stable until the delayed sync window.
  • The completed-history filter now dedupes live merged tasks against completed history instead of rendering duplicate cards.
  • The regression assertion covers the reorder branch that previously blocked same-ID ordering changes.

Negative

  • No material issues found.

@jefking jefking merged commit 1e3b370 into main Jun 2, 2026
1 of 2 checks passed
@jefking jefking deleted the moltenhub-when-tasks-go-to-grey-due-to-them-being branch June 2, 2026 00:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants