bpmn-io / bpmn-js-properties-panel

A properties panel for bpmn-js.
MIT License
290 stars 196 forks source link

STYLE CDN IS NOT WORKING: bpmn-js-properties-panel/v/1.3.0 effected css issue #961

Closed shuhaib-aot closed 1 year ago

shuhaib-aot commented 1 year ago

Describe the Bug

am currently utilizing "bpmn-js-properties-panel": "^1.3.0" in my project. However, it appears that the CSS links I had been relying on are no longer functional. Specifically, the following links are resulting in 404 errors:

Original: <link rel="stylesheet" href="https://unpkg.com/bpmn-js-properties-panel/dist/assets/properties-panel.css">
Original: <link rel="stylesheet" href="https://unpkg.com/bpmn-js-properties-panel/dist/assets/element-templates.css">

It has come to my attention that there have been updates to the CDN, but it seems that the existing version's CDNs have been affected. I would like to suggest that maintaining the availability of CDNs for existing versions is a best practice that could be considered for situations like this. image

These links are directly to latest version

image

marstamm commented 1 year ago

Hi @shuhaib-aot , when using the CDN without a version, it will fetch assets for the latest release. In the latest major release, we removed the CSS files (cf. Changelog).

Please change your CSS links to relate to the version of the properties panel you are using by adding a version tag

- <link rel="stylesheet" href="https://unpkg.com/bpmn-js-properties-panel/dist/assets/properties-panel.css">
- <link rel="stylesheet" href="https://unpkg.com/bpmn-js-properties-panel/dist/assets/element-templates.css">
+ <link rel="stylesheet" href="https://unpkg.com/bpmn-js-properties-panel@1/dist/assets/properties-panel.css">
+ <link rel="stylesheet" href="https://unpkg.com/bpmn-js-properties-panel@1/dist/assets/element-templates.css">