Skip to content
This repository has been archived by the owner on Jul 3, 2018. It is now read-only.

Премахване на еднакъв код #19

Open
AndrewRadev opened this issue Aug 29, 2014 · 2 comments
Open

Премахване на еднакъв код #19

AndrewRadev opened this issue Aug 29, 2014 · 2 comments

Comments

@AndrewRadev
Copy link
Contributor

Действията за sign_as_html и sign_as_html_fullscreen са много подобни. Вероятно има начини общите неща да се извлекат.

@martaradeva
Copy link
Contributor

Добавих ?fullscreen=on като query parameter и сега една и съща заявка GET лови и двете, после с прост if/else рендва съответния html template.
Все още има issue защото поне при мен не се показва коректно fullscreen страницата
error fullscreen

@AndrewRadev
Copy link
Contributor Author

Кода изглежда добре. Аз по-скоро мислех да извлечем обща функция, но и това върши добра работа.

Що се отнася до бъга, той не мисля, че е свързан с тази промяна. Преди около час е направена промяна в html структурата, добавен е div с клас tabelka-wrapper, който обвива tabelka-text. Съответно fullscreen варианта не е обновен да пасне и затова сега изглежда странно. Мисля, че това е релевантния commit: 5490e34. Може да си поговориш с @darzh да го оправите :).

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

2 participants