Skip to content

Add card design_1#1

Draft
oleander-twig wants to merge 1 commit intoInfJoker:masterfrom
oleander-twig:ui
Draft

Add card design_1#1
oleander-twig wants to merge 1 commit intoInfJoker:masterfrom
oleander-twig:ui

Conversation

@oleander-twig
Copy link
Copy Markdown
Collaborator

No description provided.

Comment thread Flash_cards/HTML/flash_card.html Outdated
<br/>

<form action="путь к обработчику, который принимает данные">
<p><input type="submit" value="Correct" style="height:100px; width:150px"> &nbsp;&nbsp; <input type="submit" value="Inorrect" style="height:100px; width:150px"></p>
Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

Очепятки

Comment thread Flash_cards/HTML/flash_card.html Outdated

<style>
body {
background-image: url('pic.jpg');
Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

Что за картинка? pic.jpg - очень не говорящее название

Comment thread Flash_cards/HTML/flash_card.html Outdated
@@ -0,0 +1,92 @@
<!DOCTYPE html>
Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

Нужно было бы убрать бинари, которые .DS_Store в gitignore. Другим иметь их копии не обязательно)

@@ -0,0 +1,2 @@
PyQt5
Copy link
Copy Markdown
Owner

@InfJoker InfJoker Oct 10, 2021

Choose a reason for hiding this comment

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

не проще ли было отдельную ветку сделать, и запушить новый pr? А этот старый pr pyqt оставить в этом с одним коммитом, чтобы можно было всегда к нему вернутся. А так ты получается пушишь еще пока не нужную папку с pyqt.

Лучше так и сделать, сначала исправить текущий коммит, чтобы он ничего не делал с pyqt файлами. Потом с помощью git cherry-pick выбрать твой коммит на другой ветке и его туда там добавить. А на этой дропнуть этот коммит и залить раздельными pr-aми

@oleander-twig oleander-twig marked this pull request as draft November 5, 2021 21:35
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.

2 participants