Shopify / shopify-app-bridge

https://shopify.dev/docs/api/app-bridge
86 stars 9 forks source link

What happend to Modal App Bridge #238

Open aimproxy opened 1 year ago

aimproxy commented 1 year ago

Modal in App Bridge is no longer supported in the most recent version. I feel like modals are an important part of shopify admin and avoid a lot of back and forward page navigation!...

charlesdobson commented 1 year ago

It's currently being worked on and will be released soon! I'll update here when it's out

aimproxy commented 1 year ago

Any workaround I can get it done right now? Perhaps old version of App Bridge I can use?

charlesdobson commented 1 year ago

For now, your best bet is to use one of the App Bridge npm packages if you need the modal. https://shopify.dev/docs/api/app-bridge/previous-versions/actions/modal#react

I don't have a date that Modal will be released on the new App Bridge, but it should be within the next month or two! Just wrapping up some final API considerations.

kirillplatonov commented 1 year ago

Looking forward to it. Missing Modal is deal breaker for App Bridge 3 > 4 upgrade at the moment.

resistorsoftware commented 11 months ago

Looking forward to it. Missing Modal is deal breaker for App Bridge 3 > 4 upgrade at the moment.

ARGGGGG... if I had know this library was unusable like this, I would never have adopted it for my latest updates. I feel ripped off and have wasted hours. Grrrrr...

darrynten commented 10 months ago

Button is also missing

4.x seems like a big L

charlesdobson commented 10 months ago

@darrynten The App Bridge from npm Button was only used to create the contract for Modal and TitleBar buttons. You can just use a regular HTML button now when you use things like the Title Bar. Did you have a usage for it outside of that?

darrynten commented 10 months ago

We currently use buttons:

charlesdobson commented 9 months ago

We added Modals with custom DOM content! Check it out: https://shopify.dev/docs/api/app-bridge-library/reference/modal

mgill404 commented 9 months ago

I am having issues using ui-modal in a Remix app.

My react components seem to not call their interaction callbacks with this modal. Buttons don't call onClick for example. Any advice on how to address this issue?

for a brief example, here is a Remix route

import { Button } from "@shopify/polaris";
import { useEffect } from "react";

export default function Modal() {
  function handleOnClick(e) {
    console.log("received click event"); // does not log
  }

  useEffect(() => {
    document.getElementById("test-modal").show();
  }, []);

  return (
    <ui-modal id="test-modal" variant="max">
      <Button onClick={handleOnClick}>A Button</Button>
    </ui-modal>
  );
}
daviareias commented 8 months ago

I am having issues using ui-modal in a Remix app.

My react components seem to not call their interaction callbacks with this modal. Buttons don't call onClick for example. Any advice on how to address this issue?

This is because they are using webcomponents, but for some reason Polaris uses React, the only way that I managed to fix this is this to wrap the "ui-modal" around a ref, can probably also give a ref to ui-modal, but typescript will indicate an error:

export default function Modal() {
    const ref = useRef < HTMLDivElement > null;
    const handleOnClick = () => {
        console.log("received click event");
    };

    return (
        <div ref={ref}>
            <ui-modal id="test-modal" variant="max">
                <button onClick={handleOnClick}>A Button</button>
            </ui-modal>
        </div>
    );
}
coderlinbb commented 8 months ago

我在 Remix 应用程序中使用 ui-modal 时遇到问题。 我的反应组件似乎没有用这个模式调用它们的交互回调。例如,按钮不会调用 onClick。关于如何解决这个问题有什么建议吗?

这是因为他们使用的是 web 组件,但由于某种原因 Polaris 使用 React,我设法解决这个问题的唯一方法是将“ui-modal”包裹在 ref 周围,也可以给 ui-modal 提供引用,但打字稿会提示错误:

export default function Modal() {
    const ref = useRef < HTMLDivElement > null;
    const handleOnClick = () => {
        console.log("received click event");
    };

    return (
        <div ref={ref}>
            <ui-modal id="test-modal" variant="max">
                <Button onClick={handleOnClick}>A Button</Button>
            </ui-modal>
        </div>
    );
}

Still can't

daviareias commented 8 months ago

我在 Remix 应用程序中使用 ui-modal 时遇到问题。 我的反应组件似乎没有用这个模式调用它们的交互回调。例如,按钮不会调用 onClick。关于如何解决这个问题有什么建议吗?

这是因为他们使用的是 web 组件,但由于某种原因 Polaris 使用 React,我设法解决这个问题的唯一方法是将“ui-modal”包裹在 ref 周围,也可以给 ui-modal 提供引用,但打字稿会提示错误:

export default function Modal() {
    const ref = useRef < HTMLDivElement > null;
    const handleOnClick = () => {
        console.log("received click event");
    };

    return (
        <div ref={ref}>
            <ui-modal id="test-modal" variant="max">
                <Button onClick={handleOnClick}>A Button</Button>
            </ui-modal>
        </div>
    );
}

Still can't

I tried it here and it won't work with Polaris "Button" you have use the HTML native "button" instead.

I don't know why they want us use to use the modal API when react has it's own logic that doesn't integrate well with other technologies.

coderlinbb commented 8 months ago

I used the HTML native "button" instead,but still can't.Help!!!

maxfrigge commented 8 months ago

The "native" components are great, until they are not. When you hit a road block, you can't really do anything about it because they are encapsulated from your code.

In my experience, just switching to a legacy component (e.g. Polaris Modal) when things don't work your way is a very productive solution. You can always revisit the other approach at a later point in time.

muchisx commented 8 months ago

I'm using the Remix template for development of my app but going out of the React architecture with this web-component is rather inconvenient.

I can't even use interactive Polaris components inside that modal such as buttons with click callbacks because they don't work anymore.

Anyone here using the Remix template, what did you do to remedy this? Use the previous version of app brigde react? use deprecated polaris modals? make your own modals?

would love to know and get inspired by your ways so I can find the best solution to use Polaris inside my modals!

Michael-Gibbons commented 8 months ago

I also cannot get this modal to work. The Polaris modal component is deprecated, callbacks not working for me. And the replacement isn't working. So am I just SOL?

kaczors commented 7 months ago

@charlesdobson hi :) Is there somewhere working example - the best using standard remix/polaris template? For example how to call

shopify.modal.show('modal-id')

or how to not get VS error

Property 'ui-modal' does not exist on type 'JSX.IntrinsicElements'
muchisx commented 7 months ago

@charlesdobson hi :) Is there somewhere working example - the best using standard remix/polaris template? For example how to call

shopify.modal.show('modal-id')

or how to not get VS error

Property 'ui-modal' does not exist on type 'JSX.IntrinsicElements'

For this, just update the shopify packages in your package.json.


But to continue the actual problem, can we make somehow this component load the react needed for it to work within its iFrame if used within a React/Remix context ?

tylerhellner commented 7 months ago

The modal component itself works well and seems to be using native Browser APIs under the hood (good), but I am flabbergasted why they would build a Modal with its own internal API for state. This completely breaks how React conceptualizes state, and it's incompatible with other Polaris components. What was the reason behind the change?

Michael-Gibbons commented 7 months ago

@tylerhellner to "increase friction" to prevent "incorrect usage" https://github.com/Shopify/polaris/issues/11460

ivorpad commented 7 months ago

I'm astounded by the choice to eliminate Modal support in the newest release. Modals play a key role in improving user experience and simplifying navigation. It's disheartening that this functionality has been phased out without offering a clear substitute.

muchisx commented 7 months ago

I have then now resorted to using the Modal from @shopify/app-bridge-react, which doesn't work perfectly as it works with routes, not with children and I get this strange error while working with the remix template. https://github.com/Shopify/shopify-app-template-remix/issues/541

But so far is the only way I can have a modal and stay within the React system.

CodeChd commented 7 months ago

Does anyone have a workaround for this yet? I'm facing the issue that callbacks are not working for me.

charlesdobson commented 7 months ago

We've just released App Bridge React v4 which includes a React wrapper for the ui-modal element! This sets up the React portal for you, so you can just do

<Modal>
  <MyComponent />
  <AnotherComponent />
</Modal>

https://shopify.dev/docs/api/app-bridge-library/react-components/modal


See the migration guide for more info

handhikadj commented 7 months ago

how do I prevent ui-modal to be closed when the backdrop or the close (x) button is clicked? can't see such prop in the docs

dungfv commented 7 months ago

I noticed that many Polaris components cannot work in the new modal:

Additionally, drag & drop behaviors are also blocked in the new app bridge modal. This breaks my current application.

darrynten commented 7 months ago

@handhikadj see https://github.com/Shopify/shopify-app-bridge/issues/297 for how to listen to the event, maybe you can cancel the event or just call the open event again in the close event

hmpws commented 7 months ago

useEffect(() => { setModalOpen(true); }, []);

How do I have the modal open on page load (conditionally to display a message)? App Bridge keeps giving me error. Using remix-react.

image

agustinlaurito commented 6 months ago

I noticed that many Polaris components cannot work in the new modal:

  • Popover
  • Tooltip
  • Drop zone

Additionally, drag & drop behaviors are also blocked in the new app bridge modal. This breaks my current application.

Any update regarding this issue? I'm attempting to utilize the datepicker within the modal's popover, but it's not functioning correctly. Instead, the popover appears on the page behind the modal.

shatishdesai202 commented 6 months ago

https://github.com/Shopify/shopify-app-bridge/issues/331 this one is also an issue with the new app-bridge version (v 4.1.2)

noelgoodday commented 4 months ago

I noticed that many Polaris components cannot work in the new modal:

  • Popover
  • Tooltip
  • Drop zone

Additionally, drag & drop behaviors are also blocked in the new app bridge modal. This breaks my current application.

Any update regarding this issue? I'm attempting to utilize the datepicker within the modal's popover, but it's not functioning correctly. Instead, the popover appears on the page behind the modal.

We're experiencing the same issues with drag & drop in the new AppBridge modal. I opened a dedicated issue here: https://github.com/Shopify/shopify-app-bridge/issues/364