Skip to content

Commit

Permalink
SDA-4231 (Fix button width for macOS) (finos#1900)
Browse files Browse the repository at this point in the history
* SDA-4231 - Fix button width for macOS

Signed-off-by: Kiran Niranjan <kiran.niranjan@symphony.com>

* SDA-4231 - Fix button width for macOS

Signed-off-by: Kiran Niranjan <kiran.niranjan@symphony.com>

---------

Signed-off-by: Kiran Niranjan <kiran.niranjan@symphony.com>
  • Loading branch information
KiranNiranjan authored and NguyenTranHoangSym committed Aug 15, 2023
1 parent f2208d1 commit f5944f8
Show file tree
Hide file tree
Showing 2 changed files with 14 additions and 2 deletions.
2 changes: 2 additions & 0 deletions src/renderer/components/call-notification.tsx
Original file line number Diff line number Diff line change
@@ -1,6 +1,7 @@
import classNames from 'classnames';
import { ipcRenderer } from 'electron';
import * as React from 'react';
import { isMac } from '../../common/env';
import {
darkTheme,
getContainerCssClasses,
Expand Down Expand Up @@ -125,6 +126,7 @@ export default class CallNotification extends React.Component<
false,
);
let containerCssClass = `container ${themeClassName} `;
customCssClasses.push(isMac ? 'mac' : 'windows');
containerCssClass += customCssClasses.join(' ');

const acceptText = acceptButtonText
Expand Down
14 changes: 12 additions & 2 deletions src/renderer/styles/call-notification.less
Original file line number Diff line number Diff line change
Expand Up @@ -26,6 +26,16 @@
--profile-place-holder-text: @electricity-ui-40;
}

.mac {
--button-width: 80px;
--button-with-other: 104px;
}

.windows {
--button-width: 72px;
--button-with-other: 94px;
}

body {
margin: 0;
}
Expand Down Expand Up @@ -226,7 +236,7 @@ text {
display: flex;
padding: 4px 12px;
justify-content: center;
width: 72px;
width: var(--button-width);
align-items: center;
gap: 8px;
border-radius: 16px;
Expand All @@ -249,7 +259,7 @@ text {
}

.call-type-other {
width: 94px;
width: var(--button-with-other);
}
}

Expand Down

0 comments on commit f5944f8

Please sign in to comment.