Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feature: changed styles for settings and added new call-end-sound block [WTEL-4668] #700

Merged
merged 7 commits into from
Jul 9, 2024

Conversation

liza-pohranichna
Copy link
Contributor

No description provided.

flex-direction: column;
min-width: 200px;
gap: var(--spacing-sm);

//.content-title {
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

коммент)

@@ -227,11 +252,27 @@ export default {
<style lang="scss" scoped>
.settings-section {
display: flex;
flex: 0 1 50%;
max-height: 700px;
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

а навіщо висоту обмежувати?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Обмеження висоти відбувається для того, щоб візуально блоки вибудовувались так, як на макеті) щиро кажучи, мені дуже подобається, як сторінка виглядає на макеті і, враховуючи, що сторінка сеттінгів не є динамічною (тобто блоки в ній не змінюються часто), то через це я обрала саме таке рішення.
Мені щиро здається візуально гарним цей варіант в реалізації

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

це якийсь пісєтс кринж
треба це узгоджувати з Женею

@@ -147,6 +164,7 @@ export default {

created() {
this.restoreLanguage();
this.callEndSound = !!localStorage.getItem('callEndSound');
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

а чому це не зробити одразу коли ми в даті його оглошуємо?

value
? localStorage.setItem('callEndSound', 'true')
: localStorage.removeItem('callEndSound');
this.callEndSound = value;
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

а чому тут і присвоєння, і зверху в-модел?)

@@ -227,11 +252,27 @@ export default {
<style lang="scss" scoped>
.settings-section {
display: flex;
flex: 0 1 50%;
max-height: 700px;
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

це якийсь пісєтс кринж
треба це узгоджувати з Женею

@@ -209,6 +226,13 @@ export default {
}
},

changeCallEndSoundState(value) {
value
? localStorage.setItem('callEndSound', 'true')
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

settings/callEndSound

@dlohvinov dlohvinov merged commit 1add1a9 into master Jul 9, 2024
1 check failed
@dlohvinov dlohvinov deleted the feature/call-end-setting branch July 9, 2024 08:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants