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

feat: CV/Job offer form #126

Merged
merged 14 commits into from
Sep 11, 2023
Merged

feat: CV/Job offer form #126

merged 14 commits into from
Sep 11, 2023

Conversation

HugeLetters
Copy link
Contributor

@HugeLetters HugeLetters commented Sep 8, 2023

Closes #79
Closes #80
Closes #89
Closes #39
Closes #38

patched sveltekit to include tailwind config in tsconfig file
added skeleton tailwind support
@@ -1,3 +1,3 @@
{
"tailwindCSS.experimental.classRegex": [["class:([^=]*)"]]
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 Author

Choose a reason for hiding this comment

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

Добавил небольшой патч, чтоб СвелтКит генерил тсконфиг с включенным тейлвинд конфигом - либо так, либо руками копировать инклюды из сгенеренного свелткитом тсконфига. так проще и надежнее имхо

background-color: theme('colors.white/0.1');
width: theme('width.2');
}
::selection {
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 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.

Чтобы не потерять сделал задачку #142

extend: {}
extend: {
colors: {
app: {
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Если мы прям стопроц определимся с палитрой и все с этим будет строго - можно будет перезаписать палитру телйвинда и позволить юзать только нашу.
Пока что мне показалось удобнее цвета из фигмы добавить в дополнение к дефолтной палитре - чтобы не путать цвета из тейлвинда и наши, сделал префикс "app"

},
plugins: [
plugin(({ addVariant }) => {
addVariant('current', ['&:focus-within', '&:hover']);
})
}),
skeleton
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 Author

Choose a reason for hiding this comment

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

Пропатчил пока компонент с Автокомплитом, чтобы тип значения из ивента выбора из списка совпадал с переданным значением в инпут.

  • Убрал возможность рендерить голый хтмл в лейбл скилла - станная какая-то фича у этой либы. Во-первых, это плохо, тк леблы у нас вводят юзеры, а во-вторых, был из-за этого какой-то странный баг с некорретным отображением отфильтрованных лейблов на 1 рендер

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 Author

Choose a reason for hiding this comment

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

Когда до этого патчил опенапи допустил баг - исправил

@HugeLetters HugeLetters changed the title Skillset input form feat/CV/Job offer form Sep 9, 2023
@HugeLetters HugeLetters changed the title feat/CV/Job offer form feat: CV/Job offer form Sep 9, 2023
Home
</h1>
<a
href="/hello"
Copy link
Contributor

Choose a reason for hiding this comment

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

кажется ссылочка теряется(

min:
<input
type="number"
min="0"
Copy link
Contributor

Choose a reason for hiding this comment

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

Может быть по примеру ниже поставить максимальное значение?

@HugeLetters HugeLetters merged commit 0a40941 into main Sep 11, 2023
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
2 participants