haydenull / logseq-plugin-git

A git plugin for logseq
MIT License
115 stars 24 forks source link

All functions directly present instead of single button #1

Closed ARAWKMK closed 2 years ago

ARAWKMK commented 2 years ago

Hi, I was trying your git plugin and saw that instead of single git icon, shown in your demo image,

image

directly all the options are displayed which really clutters the logseq ui

image.

CharlesChiuGit commented 2 years ago

I have similar issues. Seems like the author use the mac layout which is incompatible with windows layout.

haydenull commented 2 years ago

Sorry that Windows has not been tested since the release, version 1.0.2 should fix this issue.

CharlesChiuGit commented 2 years ago

@haydenull The new version does fix it! Thanks!

CharlesChiuGit commented 2 years ago

@haydenull But one small problem, the pop-out pane won't collapse no matter where I clicked. image

haydenull commented 2 years ago

Currently, it can only be closed by clicking a button in popup, such as show log.

CharlesChiuGit commented 2 years ago

Cool! Noted.

JohannesAck commented 2 years ago

In case somebody else has this issue: I was having the same issue as @CharlesChiuGit with the newest version of your plugin, but with the older Logseq v0.6.6.

Updating logseq to v.0.6.9 fixed the issue and now it works great.

Thanks a lot @haydenull !

ARAWKMK commented 2 years ago

Multiple buttons insted of single icon issue is resolved. But still facing same issue as faced by @CharlesChiuGit that the pop-out pane won't collapse no matter where I click. Other plugins icons have this feature. As you informed we can click on any button and it does that work and collapses the pop-out pane.

Windows 10, Logseq v0.7.0, logseq-git-plugin v1.1.0

haydenull commented 2 years ago

In case somebody else has this issue: I was having the same issue as @CharlesChiuGit with the newest version of your plugin, but with the older Logseq v0.6.6.

Updating logseq to v.0.6.9 fixed the issue and now it works great.

Thanks a lot @haydenull !

Yes, logseq has modified the html structure of header in the new version.