Skip to content

Added Flat View & Tree View Switch - #18

Closed
bryanthmr wants to merge 1 commit into
idemia:mainfrom
bryanthmr:main
Closed

bryanthmr wants to merge 1 commit into
idemia:mainfrom
bryanthmr:main

Conversation

@bryanthmr

Copy link
Copy Markdown
Contributor

You can now select a tree view for your repos to have a better vision of your repos composition.
Added icons to see if the component is a :

  • Repo (classic git icon)
  • Submodules Repo (black git icon)
  • Folder / not a repo (folder icon)

@bryanthmr
bryanthmr force-pushed the main branch 4 times, most recently from 20db5f7 to 8388c15 Compare May 11, 2026 12:35
Context Menu disabled for folder in Tree View

mypy errors fixed for linux tests
Comment thread src/mg_repo_tree.py
self.setColumnWidth(0, 300)
self.setColumnWidth(1, 40)
self.setIndentation(15) # indentation for the TreeView

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Please use the constants when refereing to the columns, not some direct numbers

Comment thread src/mg_repo_tree.py
self.headerItem().setText(0, "Git Repo Path")
self.headerItem().setText(1, "")

self.setSelectionMode(QAbstractItemView.SelectionMode.ExtendedSelection)

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Please use the constants when refereing to the columns, not some direct numbers

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

And if you need to change the columns meaning, you need to do it in mg_const.py with the COL_TITLES

Comment thread src/mg_repo_tree.py
self.header().setSortIndicatorShown(True)
self.headerItem().setToolTip(mg_const.COL_UPDATE, mg_const.MSG_TOOLTIP_UPDATE)
self.headerItem().setToolTip(1, mg_const.MSG_TOOLTIP_UPDATE)
self.headerItem().setToolTip(mg_const.COL_STATUS, mg_const.MSG_TOOLTIP_STATUS)

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Please use the constants when refereing to the columns, not some direct numbers

Comment thread src/mg_repo_tree.py

if item is None or not hasattr(item, 'repoInfo'):
dbg('Clic droit sur un dossier ou dans le vide : menu ignoré')
return

@pfremy pfremy Sep 27, 2026 •

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

text should be in english !

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

why do we need to ignore such clicks ?

@pfremy

pfremy commented Sep 27, 2026

Copy link
Copy Markdown
Collaborator

I did a first pass on comments. I need to review a bit more.

@bryanthmr please correct the first issues and update the PR. I'm curious also why the CI is failing

@pfremy

pfremy commented Oct 24, 2026

Copy link
Copy Markdown
Collaborator

PR has been reworked as part of #20

@pfremy pfremy closed this Oct 24, 2026
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