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

Добавен линк към списъка със сайтове ползван от приставката. #19

Conversation

antitoxic
Copy link
Contributor

Preview:
Preview

@antitoxic
Copy link
Contributor Author

От идеята тук: #6

@ignisf
Copy link
Member

ignisf commented Jul 3, 2013

Hi,
А защо трябва да е обвито в <ul><li>? Някакви съображения за скейлъбилити ли имаш?

@mitio
Copy link
Member

mitio commented Jul 4, 2013

Тони, мерси! Крайно време е това да го вкараме в разширението.

Аз бих махнал думата "Справки:" и бих оставил само линка. Съгласен съм с Петко, че само едно <p> ще е достатъчно. Имаш ли съображения да е <ul>?

А, и ми се струва, че има някакво разминаване в идентацията (поне така ми изглежда от diff-а).

@antitoxic
Copy link
Contributor Author

1 път да пробвам да пиша чрез мейл тук и github notification server-a да им е спукан, нищо де, от вчера:

Под ul е, понеже за мен това са references. А те само ще се умножават. Освен ако не мислим там, на 1 линк да е всичко? Но, предполагам ще има линкове и към други организации или държавни институции.

Понеже е просто "warning" страничка, не мисля, че е много важна семантиката, така че махането на "Справки" и линк в параграф ше си е ок. По-навик го щраках по-семантично. Дори щях да сменям на dl, dt+dd.

gsamokovarov added a commit that referenced this pull request Feb 4, 2014
…s-reference-link

Добавен линк към списъка със сайтове ползван от приставката.
@gsamokovarov gsamokovarov merged commit 3bbd5ff into openmediabg:master Feb 4, 2014
@ignisf
Copy link
Member

ignisf commented Feb 4, 2014

На линия съм, ако искате да пуснем нова версия

@gsamokovarov
Copy link
Contributor

Ами аз изках да дооправя забележките. Ако искате да го оставим за утре, днес нямам много ментална енергия за release :)

@ignisf
Copy link
Member

ignisf commented Feb 4, 2014

👍

@gsamokovarov
Copy link
Contributor

И да не забравим да благодарим на @antitoxic за усилията :)

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.

4 participants