ob-table-enhancer icon indicating copy to clipboard operation
ob-table-enhancer copied to clipboard

floating pane doesn't show up

Open Morganwan90 opened this issue 2 years ago • 39 comments
trafficstars

version 1.4.1 the floating pane doesn't show up when right click

截图_20230731150233

Morganwan90 avatar Jul 31 '23 07:07 Morganwan90

Same here, incredibly useful add-in otherwise. I think it went with Obsidian v 1.4.2

aikruger avatar Aug 03 '23 20:08 aikruger

same here

ChenxyLindsay avatar Aug 04 '23 13:08 ChenxyLindsay

1.4.5 same here

chenxin0397 avatar Sep 04 '23 14:09 chenxin0397

1.4.5 the same error 😂

loveminimal avatar Sep 07 '23 04:09 loveminimal

1.4.5 the same error...

Ohxbong avatar Sep 08 '23 05:09 Ohxbong

I am using Table Enhancer v0.5.10 & Obsidian v1.4.5 with the same problem

what's going on? the problem is I cannot find

A: Make sure "Settings - Appearance - Native menus" is DISABLED!

Xelbayria avatar Sep 08 '23 05:09 Xelbayria

+1

itangqiao avatar Sep 08 '23 13:09 itangqiao

+1

xjtusjtu avatar Sep 14 '23 03:09 xjtusjtu

+1

9527dong avatar Sep 20 '23 08:09 9527dong

+1

Guido-Hwang avatar Sep 20 '23 09:09 Guido-Hwang

1.4.5 +1

god please

daxiass avatar Sep 22 '23 09:09 daxiass

1.4.12 +1

UnknowSelf avatar Sep 22 '23 10:09 UnknowSelf

1.4.12 +1

Lin-Ye-Ye avatar Sep 23 '23 11:09 Lin-Ye-Ye

could someone fix this?

GedRelay avatar Sep 26 '23 10:09 GedRelay

version: 1.4.14 same problem

WalkLater avatar Oct 05 '23 03:10 WalkLater

+1 could someone fix this?

avraam-inside avatar Oct 06 '23 10:10 avraam-inside

@avraam-inside test with https://github.com/Stardusten/ob-table-enhancer/pull/98 @Stardusten please review https://github.com/Stardusten/ob-table-enhancer/pull/98

zhaihao avatar Oct 08 '23 10:10 zhaihao

@zhaihao

请问可以提供编译后的js代码段吗?

这段js代码无效

    this.registerEvent(this.app.workspace.on("layout-change", function () {
      const view = this.app.workspace.getActiveViewOfType(MarkdownView).contentEl;
      view.querySelectorAll("div.cm-embed-block.cm-table-widget").forEach(function (el) {
        el.attributes.getNamedItem("contenteditable").value = "true";
      });
    }));

C1og avatar Oct 08 '23 10:10 C1og

I tried both code block options - it doesn't work(

@zhaihao - I stuck your piece of code into compiled "main.js" file of the plugin - but the plugin simply does not start because of the line "view: HTMLElement". Without it - plugin runs, but doesn't work.

@Yzzzzzzzzzzzz Your option runs the plugin, but contenteditable is still "false" image

avraam-inside avatar Oct 08 '23 11:10 avraam-inside

It is not possible to get into this event through debugger - do you have any ideas why it does not work when the plugin is launched?

avraam-inside avatar Oct 08 '23 12:10 avraam-inside

I tried both code block options - it doesn't work(

@zhaihao - I stuck your piece of code into compiled "main.js" file of the plugin - but the plugin simply does not start because of the line "view: HTMLElement". Without it - plugin runs, but doesn't work.

@Yzzzzzzzzzzzz Your option runs the plugin, but contenteditable is still "false" image

ignore ts compile error // @ts-ignore

zhaihao avatar Oct 08 '23 13:10 zhaihao

please try to replace main.js and reload obsidian

@Yzzzzzzzzzzzz @avraam-inside

main.js.zip

zhaihao avatar Oct 08 '23 13:10 zhaihao

I could help out with running tests on the new code to see if it works or not. let me know.

Xelbayria avatar Oct 09 '23 03:10 Xelbayria

Fixed in the latest release https://github.com/Stardusten/ob-table-enhancer/releases/tag/0.5.11. Big thanks to @zhaihao

Stardusten avatar Oct 09 '23 13:10 Stardusten

@Stardusten

Yes, but what have you released?) We have been in correspondence here above - no solution has been found, and the proposed ones do not work. I downloaded the release - still hanging contenteditable = false

image

avraam-inside avatar Oct 09 '23 14:10 avraam-inside

☝️ Can confirm this on my side. Having the same problem.

Xelbayria avatar Oct 09 '23 21:10 Xelbayria

☝️ Can confirm this on my side. Having the same problem.

@Stardusten

Yes, but what have you released?) We have been in correspondence here above - no solution has been found, and the proposed ones do not work. I downloaded the release - still hanging contenteditable = false

image

My bad. My current workaround doesn't work all the time. I still need some time to figure out how to fix it.

Stardusten avatar Oct 10 '23 00:10 Stardusten

@Stardusten No worries. Take your time. The problem is a challenge to overcome. Again, let me know when your fix is ready for testing so I can test it on my side.

Xelbayria avatar Oct 10 '23 02:10 Xelbayria

☝️ Can confirm this on my side. Having the same problem.

@Stardusten Yes, but what have you released?) We have been in correspondence here above - no solution has been found, and the proposed ones do not work. I downloaded the release - still hanging contenteditable = false image

My bad. My current workaround doesn't work all the time. I still need some time to figure out how to fix it.

@Stardusten "layout-change" event only works when we change from reading view to living view. So, if we click the "edit this block" button, the DOM will be re-rendered, and contenteditable will be set to false again. I can't find any API for this event. The "editor-change" event maybe work well but has poor performance.

zhaihao avatar Oct 10 '23 06:10 zhaihao

☝️ Can confirm this on my side. Having the same problem.

@Stardusten Yes, but what have you released?) We have been in correspondence here above - no solution has been found, and the proposed ones do not work. I downloaded the release - still hanging contenteditable = false image

My bad. My current workaround doesn't work all the time. I still need some time to figure out how to fix it.

@Stardusten "layout-change" event only works when we change from reading view to living view. So, if we click the "edit this block" button, the DOM will be re-rendered, and contenteditable will be set to false again. I can't find any API for this event. The "editor-change" event maybe work well but has poor performance.

I think pop out a menu when click a table cell would be better. But that may take some times.

Stardusten avatar Oct 10 '23 06:10 Stardusten