dockge icon indicating copy to clipboard operation
dockge copied to clipboard

Improved stack list ui when using agents

Open maca134 opened this issue 7 months ago • 7 comments

⚠️⚠️⚠️ Since we do not accept all types of pull requests and do not want to waste your time. Please be sure that you have read pull request rules: https://github.com/louislam/dockge/blob/master/CONTRIBUTING.md

Tick the checkbox if you understand [x]:

  • [x] I have read and understand the pull request rules.

Description

Small change to make the stack list UI easier to use when using multiple agents.

Type of change

  • User interface (UI)

Checklist

  • [x] My code follows the style guidelines of this project
  • [x] I ran ESLint and other linters for modified files
  • [x] I have performed a self-review of my own code and tested it
  • [x] I have commented my code, particularly in hard-to-understand areas (including JSDoc for methods)
  • [x] My changes generate no new warnings
  • [ ] My code needed automated testing. I have added them (this is optional task)

Screenshots

Both agents open: c02fFOsqbSOtahQ3zWd0

One agent collapsed: PbTUuEKanPtlJxeM6DrZ

maca134 avatar May 15 '25 07:05 maca134

@maca134 sorry to bother you but any chance you'd be willing to refactor this to use endpointdisplay in StackList.vue similar to StackListItem.vue where you display the url of agents with the dropdown? Currently it uses agent.endpoint which will only display the url. endpointdisplay on the other hand especially when combined with #414 and friendly names displays the name first or the url if it has no name.

This appears to be the more intended way to get the "label" for an agent as it already existed before friendly nanes and was updated to accomodate.

I attempted half a dozen ways myself but I've also been working 12hr days. I think I'm just making a logical error since it should have been simple but anything I try ends up with a blank name tag for anything other then current.

Dracrius avatar Jun 04 '25 03:06 Dracrius

I will have a look but I have to admit this was the first time I have ever done anything in Vue.

maca134 avatar Jun 04 '25 11:06 maca134

I will have a look but I have to admit this was the first time I have ever done anything in Vue.

That is further than my understanding LOL...I know just enough to make minor fixes. I have a fork where I merged several what seems like useful PRs and I just tried to merge yours but there are conflicts in my StackList.vue. Anyways, it is probably somewhat related to what @Dracrius mentions since I do have #414 merged as well. Anyways, once I think I resolved it the entire list in the left pane goes missing.

cmcooper1980 avatar Jun 04 '25 22:06 cmcooper1980

I really want this feature.

jearton avatar Jun 05 '25 09:06 jearton

Finally fixed. I personally also prefer to keep the sub title of the agent if it isn't on the current. Seems silly but with longer lists if the agent title is scrolled off screen it improves readability. Might be worth adding a setting for before @louislam gets backs. There will be an impressive number of pulls for 1.6.0 and I think this is far cleaner then #593 but adding agent labels as a setting would really polish off this PR.

image

Dracrius avatar Jun 05 '25 22:06 Dracrius

This really needs to be added. It's so messy to manage multiple projects across multiple agents on Dockge. This would be absolutely god-tier to have.

NATroutter avatar Sep 28 '25 19:09 NATroutter

merged to my fork along with @Dracrius QOL tweak commit

cmcooper1980 avatar Oct 14 '25 00:10 cmcooper1980