Skip to content

Chapter 07 complex loops exam problems #11

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

Merged
merged 12 commits into from
Oct 9, 2018

Conversation

Antoveravip
Copy link
Collaborator

Done. Ready for check and merge.

@Antoveravip Antoveravip requested a review from vncpetrov July 22, 2018 22:09
Copy link
Collaborator

@miglen miglen left a comment

Choose a reason for hiding this comment

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

assets/chapter-7-2-images/01.stupid-password-generator-2.png - шрифта изглежда много малък докато assets/chapter-7-2-images/01.stupid-password-generator-3.png изглежда много голям.

Освен картинките, не само тези горе но и останалите ми се виждат с различна големина останалото е окей. Също забелязах, че ползваш tabs вместо spaces, според pep8 те са предпочитани моля те промени особено големите картинки със spaces като stupid password generator 2.png

@Antoveravip
Copy link
Collaborator Author

Странно как ти се виждат различни, когато са с еднакъв шрифт. :( но ще ги погледна пак.
Ще заменя табовете със спейсове и ще ги снимам наново.

@Antoveravip
Copy link
Collaborator Author

Помежду другото не съм и използвал табове, а спейсове. Може да видиш файловете и кода и да ги отвориш в Notepad++ примерно и да избереш да се виждат всички всички символи.
Ще погледна отново шрифта и ще ги направя наново снимките - за всеки случай, макар, че се съмнявам да има разлика.

@Antoveravip
Copy link
Collaborator Author

Done. Изглежда проблема идваше от широчината на определени снимки и се получаваше auto-resize, който намаляваше визуално снимката. Вече всичко би трябвало да е наред.

@vncpetrov vncpetrov merged commit 44c0bb5 into master Oct 9, 2018
@nakov nakov deleted the chapter-07-complex-loops-exam-problems branch June 21, 2019 19:02
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.

3 participants