Skip to content

Make the branch switching list table row visually incorporated into the row for the plugin/theme it is for...#898

Merged
afragen merged 3 commits intoafragen:developfrom
pbiron:develop
Sep 8, 2020
Merged

Make the branch switching list table row visually incorporated into the row for the plugin/theme it is for...#898
afragen merged 3 commits intoafragen:developfrom
pbiron:develop

Conversation

@pbiron
Copy link
Copy Markdown
Contributor

@pbiron pbiron commented Sep 6, 2020

..., just like normal update rows,

Also adds the git host icon to the branch switching row in the same place where normal update rows have the "recycle" icon, to make it more clear that that row is added by GHU.

With this PR applied, the plugins screen might look like:

gnu-branch-swithing

…he row for the plugin/theme it is for, just like normal update rows.

Also adds the git host icon to the branch switching row in the same place where normal update rows have the "recycle" icon, to make it more clear that that row is added by GHU.
@pbiron
Copy link
Copy Markdown
Contributor Author

pbiron commented Sep 6, 2020

Andy, this avoids the problem we discussed on Slack where the "branch switching row" would be updated when applying a shinny update :-)

Comment thread src/GitHub_Updater/Base.php Outdated
Comment thread src/GitHub_Updater/Base.php Outdated
@afragen
Copy link
Copy Markdown
Owner

afragen commented Sep 8, 2020

I found an edge case. The row line appears when a plugin is active and my Local Development plugin is active with it.

screenshot_133

I'm not sure what the issue is. Confirmed in both Safari and Chrome.

@afragen afragen merged commit e442cfe into afragen:develop Sep 8, 2020
@afragen
Copy link
Copy Markdown
Owner

afragen commented Sep 8, 2020

Thanks Paul!

afragen added a commit that referenced this pull request Sep 8, 2020
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