Skip to content

Commit

Permalink
Merge branch 'master' into e.muhamethanov/7696/fix-empty-value-logic
Browse files Browse the repository at this point in the history
  • Loading branch information
EldarMuhamethanov committed Oct 30, 2024
2 parents ba073a2 + 792e571 commit d34ab47
Show file tree
Hide file tree
Showing 61 changed files with 272 additions and 861 deletions.
6 changes: 3 additions & 3 deletions docs/CONTRIBUTING.md
Original file line number Diff line number Diff line change
Expand Up @@ -42,7 +42,7 @@ git config blame.ignoreRevsFile .git-blame-ignore-revs
> Не используем композицию, т.к. в ней нет необходимости,
> а также в будущем она может усложнить переход на другое решение.
- CSS-классы должны быть в формате camelCase: `elementNameModification`. [Гайд по написанию стилей](https://github.com/VKCOM/VKUI/blob/master/docs/CSS_GUIDE.md)
- CSS-классы должны быть в формате camelCase: `elementNameModification`. [Гайд по написанию стилей](CSS_GUIDE.md)
- Свойства `className` и `style` навешиваются на корневой элемент компонента
- Свойства, не используемые в коде компонента, навешиваются на **главный** элемент компонента. По умолчанию главным является корневой элемент:

Expand Down Expand Up @@ -83,10 +83,10 @@ git config blame.ignoreRevsFile .git-blame-ignore-revs
- Для цветов, скруглений, размеров, отступов и теней используются css-переменные из [vkui-tokens](https://github.com/VKCOM/vkui-tokens)
- Для типографии используются компоненты [Typography](https://vkcom.github.io/VKUI/#!/Typography) там, где это возможно
- Добавлен `export` компонента и его свойств в `packages/vkui/src/index.ts`
- Компонент покрыт юнит- и скриншотными тестами. [Гайд по тестированию](https://github.com/VKCOM/VKUI/blob/master/docs/TESTING.md)
- Компонент покрыт юнит- и скриншотными тестами. [Гайд по тестированию](TESTING.md)
- Компонент корректно отображается на всех платформах, размерах и цветовых схемах. В styleguide для всех этих параметров есть переключатели
- Код корректно работает на [поддерживаемых нами браузерах](https://github.com/VKCOM/VKUI#%D0%B1%D1%80%D0%B0%D1%83%D0%B7%D0%B5%D1%80%D1%8B)
- Для поддержки адаптивности следует придерживаться [гайда по написанию адаптивных компонентов](https://github.com/VKCOM/VKUI/blob/master/docs/ADAPTIVITY_GUIDE.md)
- Для поддержки адаптивности следует придерживаться [гайда по написанию адаптивных компонентов](ADAPTIVITY_GUIDE.md)
- `a11y` (см. пример хорошего PR с внедрением доступности, на который можно равняться [#3337](https://github.com/VKCOM/VKUI/issues/3337)):

- Компонент соответствует требованиям `a11y`
Expand Down
4 changes: 2 additions & 2 deletions docs/MAINTAINERS.md
Original file line number Diff line number Diff line change
Expand Up @@ -2,7 +2,7 @@

Данный документ содержит базовую информацию по сопровождению и процессам разработки и будет полезен в первую очередь
сопровождающим (maintainers) репозитория **VKUI**. Информацию по правилам работы с кодом можно найти в
[требованиях к разработке](https://github.com/VKCOM/VKUI/blob/master/docs/CONTRIBUTING.md).
[требованиях к разработке](CONTRIBUTING.md).

## Глоссарий

Expand Down Expand Up @@ -160,7 +160,7 @@
дедупликацию зависимостей в ветке. Если безопасно вносить изменения в ветку напрямую, то можно отметить `push directly to the branch`,
иначе будет создан `PR` с изменениями в выбранную ветку. Бывает полезно запускать эту автоматизацию после обновления
зависимостей в ветке - у нас настроен [Dependabot](https://docs.github.com/en/code-security/getting-started/dependabot-quickstart-guide),
который [обновляет зависимости](https://github.com/VKCOM/VKUI/blob/master/.github/dependabot.yml) в начале каждой недели.
который [обновляет зависимости](../.github/dependabot.yml) в начале каждой недели.
- [Update screenshots](https://github.com/VKCOM/VKUI/actions/workflows/update_screens.yml): данная автоматизация
позволяет запускать обновление скриншотов в выбранной ветке вручную (если вы забыли это сделать локально, например),
а так же отвечает за обновление скриншотов в `stable`-ветках в автоматическом режиме (потому что мы не дублируем
Expand Down
31 changes: 31 additions & 0 deletions packages/codemods/src/transforms/v7/date-picker.ts
Original file line number Diff line number Diff line change
@@ -0,0 +1,31 @@
import { API, FileInfo } from 'jscodeshift';
import { getImportInfo } from '../../codemod-helpers';
import { report } from '../../report';
import { JSCodeShiftOptions } from '../../types';

export const parser = 'tsx';

export default function transformer(file: FileInfo, api: API, options: JSCodeShiftOptions) {
const { alias } = options;
const j = api.jscodeshift;
const source = j(file.source);
const { localName } = getImportInfo(j, file, 'DatePicker', alias);

if (!localName) {
return source.toSource();
}
source
.find(j.JSXElement, {
openingElement: {
name: { name: localName },
},
})
.forEach(() => {
report(
api,
`Manual changes required for ${localName}. component DatePicker was removed in v7.0.0, please use Input, Select or DateInput component`,
);
});

return source.toSource();
}
Loading
Sorry, something went wrong. Reload?
Sorry, we cannot display this file.
Sorry, this file is invalid so it cannot be displayed.
Loading
Sorry, something went wrong. Reload?
Sorry, we cannot display this file.
Sorry, this file is invalid so it cannot be displayed.
Loading
Sorry, something went wrong. Reload?
Sorry, we cannot display this file.
Sorry, this file is invalid so it cannot be displayed.
Loading
Sorry, something went wrong. Reload?
Sorry, we cannot display this file.
Sorry, this file is invalid so it cannot be displayed.
Loading
Sorry, something went wrong. Reload?
Sorry, we cannot display this file.
Sorry, this file is invalid so it cannot be displayed.
Loading
Sorry, something went wrong. Reload?
Sorry, we cannot display this file.
Sorry, this file is invalid so it cannot be displayed.
Loading
Sorry, something went wrong. Reload?
Sorry, we cannot display this file.
Sorry, this file is invalid so it cannot be displayed.
9 changes: 8 additions & 1 deletion packages/vkui/src/components/BaseGallery/BaseGallery.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -143,6 +143,13 @@ export const BaseGallery = ({
localSlides.length <= layoutState.current.slides.length ||
layoutState.current.slides[slideIndex]?.coordX !== localSlides[slideIndex]?.coordX;

const currentSlideOffsetOnCenterAlignment =
(localContainerWidth - (localSlides[slideIndex]?.width ?? 0)) / 2;
const isFullyVisible =
align === 'center'
? localLayerWidth + currentSlideOffsetOnCenterAlignment <= localContainerWidth
: localLayerWidth <= localContainerWidth;

layoutState.current = {
containerWidth: localContainerWidth,
viewportOffsetWidth: localViewportOffsetWidth,
Expand All @@ -161,7 +168,7 @@ export const BaseGallery = ({
align,
}),
slides: localSlides,
isFullyVisible: localLayerWidth <= localContainerWidth,
isFullyVisible,
};

setShiftState((prevState) => ({
Expand Down
6 changes: 0 additions & 6 deletions packages/vkui/src/components/Button/Button.module.css
Original file line number Diff line number Diff line change
Expand Up @@ -60,9 +60,6 @@
text-align: start;
}

.alignCenter {
}

.alignRight .in {
justify-content: flex-end;
text-align: end;
Expand All @@ -79,9 +76,6 @@
opacity: 0.64;
}

.withIcon {
}

.content {
white-space: nowrap;
text-overflow: ellipsis;
Expand Down
5 changes: 1 addition & 4 deletions packages/vkui/src/components/Button/Button.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -36,7 +36,6 @@ const stylesAppearance = {

const stylesAlign = {
left: styles.alignLeft,
center: styles.alignCenter,
right: styles.alignRight,
};

Expand Down Expand Up @@ -80,7 +79,6 @@ export const Button = ({
disabled,
...restProps
}: ButtonProps): React.ReactNode => {
const hasIcons = Boolean(before || after);
const hasIconOnly = !children && Boolean(after) !== Boolean(before);
const { sizeY = 'none' } = useAdaptivity();
const platform = usePlatform();
Expand All @@ -100,11 +98,10 @@ export const Button = ({
stylesSize[size],
stylesMode[mode],
stylesAppearance[appearance],
stylesAlign[align],
align !== 'center' && stylesAlign[align],
sizeY !== 'compact' && sizeYClassNames[sizeY],
platform === 'ios' && styles.ios,
stretched && styles.stretched,
hasIcons && styles.withIcon,
hasIconOnly && !stretched && styles.singleIcon,
loading && styles.loading,
rounded && styles.rounded,
Expand Down

This file was deleted.

11 changes: 0 additions & 11 deletions packages/vkui/src/components/DatePicker/DatePicker.e2e.tsx

This file was deleted.

31 changes: 0 additions & 31 deletions packages/vkui/src/components/DatePicker/DatePicker.module.css

This file was deleted.

25 changes: 0 additions & 25 deletions packages/vkui/src/components/DatePicker/DatePicker.stories.tsx

This file was deleted.

Loading

0 comments on commit d34ab47

Please sign in to comment.