Skip to content

Commit

Permalink
docs: new cem for Topic-P components - ShellBar (#7823)
Browse files Browse the repository at this point in the history
* docs: new cem for Topic-P components - ShellBar
Issue: #7610

* docs: new cem for Topic-P components - ShellBar
- fixed review comments

* fix(ui5-wizard): scrollbar styles are now present on root (#7838)

* docs: new cem for Topic-P components - ShellBar
- fixed review comments

* chore: update chromedriver to 119 (#7883)

* fix(ui5-switch): align 'off' text in RTL, add compact mode params (#7603)

In RTL mode the 'off' icon of `Graphical` type Switch was misaligned due to missing parameters for RTL scenario of the control.

Fixes: #7522 
Fixes: #7806

* fix(framework): redundant fonts loading (#7868)

fix(framework): font loading

Co-authored-by: Nayden Naydenov <[email protected]>

* docs: new cem for Topic-P components - ShellBar
- fixed review comments

---------

Co-authored-by: Ivaylo Plashkov <[email protected]>
Co-authored-by: ilhan orhan <[email protected]>
Co-authored-by: Stoyan <[email protected]>
Co-authored-by: Nayden Naydenov <[email protected]>
Co-authored-by: Nayden Naydenov <[email protected]>
  • Loading branch information
6 people authored Nov 21, 2023
1 parent 7a3954f commit 8d79551
Show file tree
Hide file tree
Showing 11 changed files with 96 additions and 170 deletions.
2 changes: 1 addition & 1 deletion packages/base/package.json
Original file line number Diff line number Diff line change
Expand Up @@ -38,7 +38,7 @@
"@buxlabs/amd-to-es6": "0.16.1",
"@openui5/sap.ui.core": "1.116.0",
"@ui5/webcomponents-tools": "1.20.0-rc.1",
"chromedriver": "118.0.1",
"chromedriver": "119.0.1",
"clean-css": "^5.2.2",
"copy-and-watch": "^0.1.5",
"cross-env": "^7.0.3",
Expand Down
8 changes: 5 additions & 3 deletions packages/base/src/ManagedStyles.ts
Original file line number Diff line number Diff line change
Expand Up @@ -105,16 +105,18 @@ const updateStyle = (data: StyleData, name: string, value = "", theme?: string)
}
};

const hasStyle = (name: string, value = "") => {
const hasStyle = (name: string, value = ""): boolean => {
if (shouldUseLinks()) {
return !!document.querySelector(`head>link[${name}="${value}"]`);
}

const styleElement = document.querySelector(`head>style[${name}="${value}"]`);

if (document.adoptedStyleSheets && !isSafari()) {
return !!document.adoptedStyleSheets.find(sh => (sh as Record<string, any>)._ui5StyleId === getStyleId(name, value));
return !!styleElement || !!document.adoptedStyleSheets.find(sh => (sh as Record<string, any>)._ui5StyleId === getStyleId(name, value));
}

return !!document.querySelector(`head>style[${name}="${value}"]`);
return !!styleElement;
};

const removeStyle = (name: string, value = "") => {
Expand Down
2 changes: 1 addition & 1 deletion packages/fiori/package.json
Original file line number Diff line number Diff line change
Expand Up @@ -49,6 +49,6 @@
},
"devDependencies": {
"@ui5/webcomponents-tools": "1.20.0-rc.1",
"chromedriver": "118.0.1"
"chromedriver": "119.0.1"
}
}
10 changes: 0 additions & 10 deletions packages/fiori/src/Interfaces.ts
Original file line number Diff line number Diff line change
Expand Up @@ -61,15 +61,6 @@ const INotificationListItem = "sap.ui.webc.fiori.INotificationListItem";
*/
const IProductSwitchItem = "sap.ui.webc.fiori.IProductSwitchItem";

/**
* Interface for components that may be slotted inside <code>ui5-shellbar</code> as items
*
* @name sap.ui.webc.fiori.IShellBarItem
* @interface
* @public
*/
const IShellBarItem = "sap.ui.webc.fiori.IShellBarItem";

/**
* Interface for components that may be slotted inside <code>ui5-side-navigation</code> as items
*
Expand Down Expand Up @@ -132,7 +123,6 @@ export {
INotificationAction,
INotificationListItem,
IProductSwitchItem,
IShellBarItem,
ISideNavigationItem,
ISideNavigationSubItem,
ISortItem,
Expand Down
Loading

0 comments on commit 8d79551

Please sign in to comment.