Skip to content

Conversation

danirabbit
Copy link
Member

No description provided.

@danirabbit danirabbit requested a review from jeremypw May 19, 2021 20:15
Copy link
Collaborator

@jeremypw jeremypw left a comment

Choose a reason for hiding this comment

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

Yes, this is a much better way of doing it - thanks!

@jeremypw jeremypw merged commit 122028a into choose-project-popover May 20, 2021
@jeremypw jeremypw deleted the danrabbit/choose-project-popover-projectliststore branch May 20, 2021 09:07
jeremypw pushed a commit that referenced this pull request May 26, 2021
* Add ChooseProjectButton class

* Add ChooseProjectButton to HeaderBar and react to add/remove project

* ChooseProjectButton.vala: Update header

* Fix lint

* Use HeaderBar active project new-branch action

* Use ChooseProjectButton active project for find-global action

* Fix closing all except

* FileView responsible for removing project from GitManager when closed

* ChooseProjectButton: Expand horizontally, add top & bottom margins

* ChooseProjectButton: Limit width to 24 chars

* ChooseProjectButton: Set max chars and ellipsize

* ChooseProjectButton: use max_content_height for scrolled (#1038)

* ChooseProjectButton: bind ProjectEntry radio to active (#1040)

* ChooseProjectButton: valign center (#1037)

* ChooseProjectButton: subclass menubutton (#1039)

* ChooseProjectButton.vala: Lose unused code

get_radio_group ()

Co-authored-by: Daniel Foré <daniel@elementary.io>

* Show project path in tooltip

* Replace missing brace

* ProjectEntry: Rename to ProjectRow

* Remove duplicated code

* Update src/Widgets/ChooseProjectButton.vala

Co-authored-by: Daniel Foré <daniel@elementary.io>

* Update src/Widgets/ChooseProjectButton.vala: Explict get/set

Co-authored-by: Daniel Foré <daniel@elementary.io>

* More concise names

* Reduce scope if `img`

* ChooseProjectButton: ellipsize long radio items

* ChooseProjectButton: use a liststore to populate project list (#1042)

* ChooseProjectButton: use a liststore to populate project list

* scope down manager

* smaller diff

* remove unused method

* Update src/Widgets/HeaderBar.vala: remove unused setter

Co-authored-by: Daniel Foré <daniel@elementary.io>

* GitManager: source of truth for active_project_path (#1044)

* ChooseProjectButton: simplify ProjectRow click handling (#1041)

* ChooseProjectButton: simplify ProjectRow click handling

* actual fix

* Allow non-git projects to fix global search (#1045)

* Remove size group

Co-authored-by: Jeremy Wootten <jeremy@elementaryos.org>
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