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

Add landing page #113

Merged
merged 19 commits into from Nov 23, 2022
Merged

Add landing page #113

merged 19 commits into from Nov 23, 2022

Conversation

niyak93rus
Copy link
Contributor

#111: make landing page

@niyak93rus niyak93rus changed the title Add landing parge Add landing pare Sep 26, 2022
@niyak93rus niyak93rus changed the title Add landing pare Add landing page Sep 26, 2022
@ashikov ashikov added enhancement New feature or request frontend labels Sep 27, 2022
@ashikov
Copy link

ashikov commented Sep 29, 2022

@kacetal привет! Глянь, плиз

<html lang="en">

<head>
<link rel="stylesheet" href="https://cdn.jsdelivr.net/npm/bootstrap@5.2.0/dist/css/bootstrap.min.css"
Copy link
Collaborator

Choose a reason for hiding this comment

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

надо использовать WebJar как это сделано в других файлах

</div>
</footer>
</div>
<script src="https://cdn.jsdelivr.net/npm/bootstrap@5.2.1/dist/js/bootstrap.bundle.min.js"
Copy link
Collaborator

Choose a reason for hiding this comment

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

webJar

@ashikov
Copy link

ashikov commented Oct 12, 2022

@niyak93rus тут бы поправить пару моментов о которых написал @kacetal

niyak93rus and others added 2 commits October 14, 2022 09:49
add bootstrap webjar to landing page and pom.xml, correct link to project's page
@niyak93rus
Copy link
Contributor Author

@ashikov @kacetal Доброго дня, внес изменения, посмотрите, как будет время пож-та

pom.xml Outdated
@@ -154,10 +154,15 @@
<version>${database-rider.version}</version>
<scope>test</scope>
</dependency>
<dependency>
Copy link
Collaborator

Choose a reason for hiding this comment

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

Bootstrap уже есть на линии 69, добавлять его еще раз не надо, может просто версии обновить на более свежую и проверить что ничего не сломалось

pom.xml Outdated
</dependencies>

<build>
<plugins>
<plugins>
Copy link
Collaborator

Choose a reason for hiding this comment

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

убери пробел

<!DOCTYPE html>
<html lang="en">

<head>
Copy link
Collaborator

Choose a reason for hiding this comment

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

используй уже готовый блок head как тут

если надо что добавить измены этот файл

Copy link
Contributor Author

Choose a reason for hiding this comment

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

если я правильно понял, то из папки static нет доступа к шаблонам thymeleaf. то есть тег у меня так просто пустой становится в итоге

Copy link
Contributor Author

Choose a reason for hiding this comment

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

@kacetal с этим не получилось, не понимаю, как правильно сделать. если я правильно понял, то из папки static нет доступа к шаблонам thymeleaf. Если я так же копирую, тег у меня пустой становится.

Copy link
Collaborator

Choose a reason for hiding this comment

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

Ну надо не использовать папку статик

Copy link
Collaborator

Choose a reason for hiding this comment

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

Клади в шаблоны

</div>
</footer>
</div>
<script src="/webjars/bootstrap/js/bootstrap.min.js"></script>
Copy link
Collaborator

Choose a reason for hiding this comment

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

in the head

@ssssank
Copy link
Contributor

ssssank commented Oct 31, 2022

@niyak93rus доделаете? :)

@niyak93rus
Copy link
Contributor Author

@ssssank да, конечно, в ближайшие дни займусь 👍

@kacetal
Copy link
Collaborator

kacetal commented Nov 14, 2022

перенеси картинки в static

@ashikov
Copy link

ashikov commented Nov 18, 2022

@niyak93rus тыц. Вроде по мелочи осталось доделать

@kacetal
Copy link
Collaborator

kacetal commented Nov 23, 2022

@niyak93rus Спасибо за помощь, я добавил изменений связанных со статическими ресурсами и контроллер для этой страницы. В остальном всё хорошо, можно мержить.

@kacetal kacetal merged commit f3e8ac7 into Hexlet:main Nov 23, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants