Skip to content
This repository has been archived by the owner on Sep 11, 2023. It is now read-only.

feat/landing-5 #20

Merged
merged 6 commits into from
Jun 2, 2023
Merged

feat/landing-5 #20

merged 6 commits into from
Jun 2, 2023

Conversation

oxiqod
Copy link
Contributor

@oxiqod oxiqod commented May 26, 2023

No description provided.

@oxiqod oxiqod self-assigned this May 26, 2023
@oxiqod oxiqod linked an issue May 26, 2023 that may be closed by this pull request
@oxiqod oxiqod changed the title feat: landing-5 feat/landing-5 May 26, 2023
@oxiqod oxiqod requested a review from TorinAsakura May 26, 2023 12:53
Copy link
Member

@TorinAsakura TorinAsakura left a comment

Choose a reason for hiding this comment

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

У меня всё

Снимок экрана 2023-05-27 в 02 07 01

а это вишенкой
Warning: forwardRef render functions accept exactly two parameters: props and ref. Did you forget to use the ref parameter?

@oxiqod oxiqod requested a review from TorinAsakura May 27, 2023 10:58
@@ -0,0 +1,4 @@
export const TechnologyKit = {
webDevelopment: 'JavaScript,HTML,CSS,SASS,Vanilla JS,Jade',
Copy link
Member

Choose a reason for hiding this comment

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

ты издеваешься? это должен быть массив объектов, по которому можно итерироваться, завтра нам надо будет выбросить хтмл, чо, пойдёшь в код, править строку? А если опечатаешься?

Copy link
Contributor Author

@oxiqod oxiqod May 28, 2023

Choose a reason for hiding this comment

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

@TorinAsakura
Таким образом? ссылка
не смогла найти среди проектов как выполняется запись элементов именно для бегущей строки

так не верно? ты сказал именно объекты в массиве
export const technologyKit = {
webDev: ['JavaScript','HTML','CSS','SASS','Vanilla JS','Jade'],
workspace: ['Github','Atom','Figma']
}

или

export const KitWebDev = ['JavaScript','HTML','CSS','SASS','Vanilla JS','Jade']
export const KitWorkspace = ['Github','Atom','Figma']

Copy link
Member

Choose a reason for hiding this comment

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

Что значит "не смогла найти как выполняется запись "ИМЕННО ДЛЯ БЕГУЩЕЙ СТРОКИ""?????

ui/condition/src/condition.component.tsx Outdated Show resolved Hide resolved
ui/condition/src/condition.component.tsx Outdated Show resolved Hide resolved
Copy link
Member

@TorinAsakura TorinAsakura left a comment

Choose a reason for hiding this comment

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

И я не увидел фиксов по неймингам

Снимок экрана 2023-05-28 в 00 23 27

Copy link
Member

@TorinAsakura TorinAsakura left a comment

Choose a reason for hiding this comment

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

в который (и последний) раз обращу твоё внимание на "некоторые" элементы требующие сверки с фигмой…

ui/condition/src/condition.interface.ts Outdated Show resolved Hide resolved
@oxiqod
Copy link
Contributor Author

oxiqod commented May 30, 2023

@TorinAsakura

в который (и последний) раз обращу твоё внимание на "некоторые" элементы требующие сверки с фигмой…

речь же о кнопке курсы из мобильной версии (перенос слова)?
или у тебя не отображается еще и вложенная в нее иконка?

@TorinAsakura
Copy link
Member

@oksssvv На скриншот смотри

@oxiqod
Copy link
Contributor Author

oxiqod commented May 30, 2023

@TorinAsakura

На скриншоте смотри

image

Судя по скрину - иконка не отображается, а другие кнопки в которых есть вложение иконки? Которые ниже
У меня просто все на месте, мне нужно понять что не так, что бы исправить

@TorinAsakura
Copy link
Member

@oksssvv у меня вопрос только к тому, что на скрине

@oxiqod oxiqod requested a review from TorinAsakura May 30, 2023 08:04
Copy link
Member

@TorinAsakura TorinAsakura left a comment

Choose a reason for hiding this comment

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

  1. Warning: Received true for a non-boolean attribute fill.

If you want to write it to the DOM, pass a string instead: fill="true" or fill={value.toString()}.
at button
at /Users/torinasakura/workspace/academy/oksssvv.github.io/.yarn/virtual/@emotion-react-virtual-b301de22a9/4/.yarn/berry/cache/@emotion-react-npm-11.11.0-fda75c579c-9.zip/node_modules/@emotion/react/dist/emotion-element-48d2c2e4.cjs.dev.js:70:25
at eval (webpack-internal:///../../../../.yarn/virtual/@ui-button-virtual-6366d4c56c/1/ui/button/src/button.component.tsx:36:83)
at div
at /Users/torinasakura/workspace/academy/oksssvv.github.io/.yarn/virtual/@emotion-react-virtual-b301de22a9/4/.yarn/berry/cache/@emotion-react-npm-11.11.0-fda75c579c-9.zip/node_modules/@emotion/react/dist/emotion-element-48d2c2e4.cjs.dev.js:70:25
at div
at /Users/torinasakura/workspace/academy/oksssvv.github.io/.yarn/virtual/@emotion-react-virtual-b301de22a9/4/.yarn/berry/cache/@emotion-react-npm-11.11.0-fda75c579c-9.zip/node_modules/@emotion/react/dist/emotion-element-48d2c2e4.cjs.dev.js:70:25
at div
at /Users/torinasakura/workspace/academy/oksssvv.github.io/.yarn/virtual/@emotion-react-virtual-b301de22a9/4/.yarn/berry/cache/@emotion-react-npm-11.11.0-fda75c579c-9.zip/node_modules/@emotion/react/dist/emotion-element-48d2c2e4.cjs.dev.js:70:25
at Navigation (webpack-internal:///../../../../.yarn/virtual/@landing-navigation-fragment-virtual-f1fc73f6a4/1/landing/fragments/landing-navigation/src/navigation.component.tsx:37:69)
at div
at /Users/torinasakura/workspace/academy/oksssvv.github.io/.yarn/virtual/@emotion-react-virtual-b301de22a9/4/.yarn/berry/cache/@emotion-react-npm-11.11.0-fda75c579c-9.zip/node_modules/@emotion/react/dist/emotion-element-48d2c2e4.cjs.dev.js:70:25
at IndexPage
at ThemeProvider (/Users/torinasakura/workspace/academy/oksssvv.github.io/.yarn/virtual/@emotion-react-virtual-b301de22a9/4/.yarn/berry/cache/@emotion-react-npm-11.11.0-fda75c579c-9.zip/node_modules/@emotion/react/dist/emotion-element-48d2c2e4.cjs.dev.js:125:32)
at ThemeProvider (webpack-internal:///../../../../.yarn/virtual/@ui-theme-virtual-8835c06af0/1/ui/theme/src/theme.provider.tsx:20:26)
at IntlProvider (/Users/torinasakura/workspace/academy/oksssvv.github.io/.yarn/virtual/react-intl-virtual-694b2aa2d7/4/.yarn/berry/cache/react-intl-npm-6.4.2-0718017b10-9.zip/node_modules/react-intl/src/components/provider.js:88:47)
at App (webpack-internal:///./pages/_app.tsx:22:16)
at StyleRegistry (/Users/torinasakura/workspace/academy/oksssvv.github.io/.yarn/virtual/styled-jsx-virtual-51128919e0/4/.yarn/berry/cache/styled-jsx-npm-5.0.7-d47abf001f-9.zip/node_modules/styled-jsx/dist/index/index.js:448:36)
at AppContainer (/Users/torinasakura/workspace/academy/oksssvv.github.io/.yarn/virtual/next-virtual-919027c60d/4/.yarn/berry/cache/next-npm-12.3.4-cdaf2db0a7-9.zip/node_modules/next/dist/server/render.js:298:29)
at AppContainerWithIsomorphicFiberStructure (/Users/torinasakura/workspace/academy/oksssvv.github.io/.yarn/virtual/next-virtual-919027c60d/4/.yarn/berry/cache/next-npm-12.3.4-cdaf2db0a7-9.zip/node_modules/next/dist/server/render.js:327:57)
at div
at Body (/Users/torinasakura/workspace/academy/oksssvv.github.io/.yarn/virtual/next-virtual-919027c60d/4/.yarn/berry/cache/next-npm-12.3.4-cdaf2db0a7-9.zip/node_modules/next/dist/server/render.js:614:21)

  1. Что ты там с этой кнопкой несчастной столько времени возишься Оксан?
2023-06-01.21.55.36.mov

@oxiqod oxiqod requested a review from TorinAsakura June 2, 2023 08:53
Copy link
Member

@TorinAsakura TorinAsakura left a comment

Choose a reason for hiding this comment

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

Полагаю, на 6-м степе мы с тобой обсудим вопросы адаптивности, так как это конечно же катастрофа, ну, с учётом количества примеров, которые у тебя есть под рукой

image

@TorinAsakura TorinAsakura merged commit b00cec6 into master Jun 2, 2023
@TorinAsakura TorinAsakura deleted the feat/landing-5 branch June 2, 2023 13:57
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Academy landing, step 5
2 participants