Skip to content

Commit

Permalink
nemesis cleanup
Browse files Browse the repository at this point in the history
  • Loading branch information
@seditio
seditio committed Dec 20, 2011
1 parent e0f78a0 commit 69fbb2f
Show file tree
Hide file tree
Showing 40 changed files with 2,115 additions and 2,216 deletions.
136 changes: 67 additions & 69 deletions modules/forums/tpl/forums.editpost.tpl
View file
Original file line number Diff line number Diff line change
@@ -1,73 +1,71 @@
<!-- BEGIN: MAIN -->

<div class="block content">
<h2 class="forums">{FORUMS_EDITPOST_PAGETITLE}</h2>
<div class="blockbody">
<!-- IF {FORUMS_EDITPOST_SUBTITLE} --><p class="marginbottom10 small">{FORUMS_EDITPOST_SUBTITLE}</p><!-- ENDIF -->

{FILE "{PHP.cfg.themes_dir}/{PHP.cfg.defaulttheme}/warnings.tpl"}

<form action="{FORUMS_EDITPOST_SEND}" method="post" name="editpost">
<table class="cells">
<!-- BEGIN: FORUMS_EDITPOST_FIRSTPOST -->
<tr>
<td class="width20">{PHP.L.forums_topic}:</td>
<td class="width80">{FORUMS_EDITPOST_TOPICTITTLE}</td>
</tr>
<tr>
<td>{PHP.L.Description}:</td>
<td>{FORUMS_EDITPOST_TOPICDESCRIPTION}</td>
</tr>
<!-- END: FORUMS_EDITPOST_FIRSTPOST -->
<tr>
<td colspan="2">{FORUMS_EDITPOST_TEXT}</td>
</tr>
<!-- BEGIN: POLL -->
<tr>
<script type="text/javascript" src="{PHP.cfg.modules_dir}/polls/js/polls.js"></script>
<script type="text/javascript">
var ansMax = {PHP.cfg.polls.max_options_polls};
</script>
<td>{PHP.L.poll}:</td>
<td>
{EDIT_POLL_IDFIELD}
{EDIT_POLL_TEXT}
</td>
</tr>
<tr>
<td>{PHP.L.Options}:</td>
<td>
<!-- BEGIN: OPTIONS -->
<div class="polloptiondiv">
{EDIT_POLL_OPTION_TEXT}
<input name="deloption" value="x" type="button" class="deloption" style="display:none;" />
</div>
<!-- END: OPTIONS -->
<input id="addoption" name="addoption" value="{PHP.L.Add}" type="button" style="display:none;" /></td>
</tr>
<tr>
<td>&nbsp;</td>
<td>{EDIT_POLL_MULTIPLE}</td>
</tr>
<!-- BEGIN: EDIT -->
<tr>
<td>&nbsp;</td>
<td>{EDIT_POLL_LOCKED} {EDIT_POLL_RESET} {EDIT_POLL_DELETE}</td>
</tr>
<!-- END: EDIT -->
<!-- END: POLL -->
<!-- BEGIN: FORUMS_EDITPOST_TAGS -->
<tr>
<td>{PHP.L.Tags}:</td>
<td>{FORUMS_EDITPOST_FORM_TAGS} ({FORUMS_EDITPOST_TOP_TAGS_HINT})</td>
</tr>
<!-- END: FORUMS_EDITPOST_TAGS -->
<tr>
<td colspan="2" class="valid"><button type="submit">{PHP.L.Update}</button></td>
</tr>
</table>
</form>
</div>
</div>
<div class="block">
<h2 class="forums">{FORUMS_EDITPOST_PAGETITLE}</h2>
<!-- IF {FORUMS_EDITPOST_SUBTITLE} --><p class="marginbottom10 small">{FORUMS_EDITPOST_SUBTITLE}</p><!-- ENDIF -->
{FILE "{PHP.cfg.themes_dir}/{PHP.cfg.defaulttheme}/warnings.tpl"}
<form action="{FORUMS_EDITPOST_SEND}" method="post" name="editpost">
<table class="cells">
<!-- BEGIN: FORUMS_EDITPOST_FIRSTPOST -->
<tr>
<td class="width20">{PHP.L.forums_topic}:</td>
<td class="width80">{FORUMS_EDITPOST_TOPICTITTLE}</td>
</tr>
<tr>
<td>{PHP.L.Description}:</td>
<td>{FORUMS_EDITPOST_TOPICDESCRIPTION}</td>
</tr>
<!-- END: FORUMS_EDITPOST_FIRSTPOST -->
<tr>
<td colspan="2">{FORUMS_EDITPOST_TEXT}</td>
</tr>
<!-- BEGIN: POLL -->
<tr>
<script type="text/javascript" src="{PHP.cfg.modules_dir}/polls/js/polls.js"></script>
<script type="text/javascript">
var ansMax = {PHP.cfg.polls.max_options_polls};
</script>
<td>{PHP.L.poll}:</td>
<td>
{EDIT_POLL_IDFIELD}
{EDIT_POLL_TEXT}
</td>
</tr>
<tr>
<td>{PHP.L.Options}:</td>
<td>
<!-- BEGIN: OPTIONS -->
<div class="polloptiondiv">
{EDIT_POLL_OPTION_TEXT}
<input name="deloption" value="x" type="button" class="deloption" style="display:none;" />
</div>
<!-- END: OPTIONS -->
<input id="addoption" name="addoption" value="{PHP.L.Add}" type="button" style="display:none;" /></td>
</tr>
<tr>
<td>&nbsp;</td>
<td>{EDIT_POLL_MULTIPLE}</td>
</tr>
<!-- BEGIN: EDIT -->
<tr>
<td>&nbsp;</td>
<td>{EDIT_POLL_LOCKED} {EDIT_POLL_RESET} {EDIT_POLL_DELETE}</td>
</tr>
<!-- END: EDIT -->
<!-- END: POLL -->
<!-- BEGIN: FORUMS_EDITPOST_TAGS -->
<tr>
<td>{PHP.L.Tags}:</td>
<td>{FORUMS_EDITPOST_FORM_TAGS} ({FORUMS_EDITPOST_TOP_TAGS_HINT})</td>
</tr>
<!-- END: FORUMS_EDITPOST_TAGS -->
<tr>
<td colspan="2" class="valid">
<button type="submit">{PHP.L.Update}</button>
</td>
</tr>
</table>
</form>
</div>

<!-- END: MAIN -->
136 changes: 67 additions & 69 deletions modules/forums/tpl/forums.newtopic.tpl
View file
Original file line number Diff line number Diff line change
@@ -1,73 +1,71 @@
<!-- BEGIN: MAIN -->

<div class="block content">
<h2 class="forums">{FORUMS_NEWTOPIC_PAGETITLE}</h2>
<div class="blockbody">
{FILE "{PHP.cfg.themes_dir}/{PHP.cfg.defaulttheme}/warnings.tpl"}
<div class="block">
<h2 class="forums">{FORUMS_NEWTOPIC_PAGETITLE}</h2>
{FILE "{PHP.cfg.themes_dir}/{PHP.cfg.defaulttheme}/warnings.tpl"}
<form action="{FORUMS_NEWTOPIC_SEND}" method="post" name="newtopic">
<table class="cells">
<tr>
<td class="width20">{PHP.L.Title}:</td>
<td class="width80">{FORUMS_NEWTOPIC_TITLE}</td>
</tr>
<tr>
<td>{PHP.L.Description}:</td>
<td>{FORUMS_NEWTOPIC_DESC}</td>
</tr>
<!-- BEGIN: PRIVATE -->
<tr>
<td>{PHP.L.forums_privatetopic1}:</td>
<td>
{FORUMS_NEWTOPIC_ISPRIVATE}
<span class="small">({PHP.L.forums_privatetopic2})</span>
</td>
</tr>
<!-- END: PRIVATE -->
<tr>
<td colspan="2">{FORUMS_NEWTOPIC_TEXT}</td>
</tr>
<!-- BEGIN: POLL -->
<tr>
<td>{PHP.L.poll}:</td>
<td>
<script type="text/javascript" src="{PHP.cfg.modules_dir}/polls/js/polls.js"></script>
<script type="text/javascript">
var ansMax = {PHP.cfg.polls.max_options_polls};
</script>
{EDIT_POLL_IDFIELD}
{EDIT_POLL_TEXT}
</td>
</tr>
<tr>
<td>{PHP.L.Options}:</td>
<td>
<!-- BEGIN: OPTIONS -->
<div class="polloptiondiv">
{EDIT_POLL_OPTION_TEXT}
<input name="deloption" value="x" type="button" class="deloption" style="display:none;" />
</div>
<!-- END: OPTIONS -->
<input id="addoption" name="addoption" value="{PHP.L.Add}" type="button" style="display:none;" /></td>
</tr>
<tr>
<td></td>
<td>
{EDIT_POLL_MULTIPLE}
</td>
</tr>
<!-- END: POLL -->
<!-- BEGIN: FORUMS_NEWTOPIC_TAGS -->
<tr>
<td>{PHP.L.Tags}:</td>
<td>{FORUMS_NEWTOPIC_FORM_TAGS} ({FORUMS_NEWTOPIC_TOP_TAGS_HINT})</td>
</tr>
<!-- END: FORUMS_NEWTOPIC_TAGS -->
<tr>
<td colspan="2" class="valid"><button type="submit">{PHP.L.Submit}</button></td>
</tr>
</table>
</form>
</div>

<form action="{FORUMS_NEWTOPIC_SEND}" method="post" name="newtopic">
<table class="cells">
<tr>
<td class="width20">{PHP.L.Title}:</td>
<td class="width80">{FORUMS_NEWTOPIC_TITLE}</td>
</tr>
<tr>
<td>{PHP.L.Description}:</td>
<td>{FORUMS_NEWTOPIC_DESC}</td>
</tr>
<!-- BEGIN: PRIVATE -->
<tr>
<td>{PHP.L.forums_privatetopic1}:</td>
<td>
{FORUMS_NEWTOPIC_ISPRIVATE}
<span class="small">({PHP.L.forums_privatetopic2})</span>
</td>
</tr>
<!-- END: PRIVATE -->
<tr>
<td colspan="2">{FORUMS_NEWTOPIC_TEXT}</td>
</tr>
<!-- BEGIN: POLL -->
<tr>
<td>{PHP.L.poll}:</td>
<td>
<script type="text/javascript" src="{PHP.cfg.modules_dir}/polls/js/polls.js"></script>
<script type="text/javascript">
var ansMax = {PHP.cfg.polls.max_options_polls};
</script>
{EDIT_POLL_IDFIELD}
{EDIT_POLL_TEXT}
</td>
</tr>
<tr>
<td>{PHP.L.Options}:</td>
<td>
<!-- BEGIN: OPTIONS -->
<div class="polloptiondiv">
{EDIT_POLL_OPTION_TEXT}
<input name="deloption" value="x" type="button" class="deloption" style="display:none;" />
</div>
<!-- END: OPTIONS -->
<input id="addoption" name="addoption" value="{PHP.L.Add}" type="button" style="display:none;" /></td>
</tr>
<tr>
<td></td>
<td>
{EDIT_POLL_MULTIPLE}
</td>
</tr>
<!-- END: POLL -->
<!-- BEGIN: FORUMS_NEWTOPIC_TAGS -->
<tr>
<td>{PHP.L.Tags}:</td>
<td>{FORUMS_NEWTOPIC_FORM_TAGS} ({FORUMS_NEWTOPIC_TOP_TAGS_HINT})</td>
</tr>
<!-- END: FORUMS_NEWTOPIC_TAGS -->
<tr>
<td colspan="2" class="valid"><button type="submit">{PHP.L.Submit}</button></td>
</tr>
</table>
</form>
</div>
</div>
<!-- END: MAIN -->
Loading

17 comments on commit 69fbb2f

@esclkm
Copy link
Contributor

@esclkm esclkm commented on 69fbb2f Dec 22, 2011

Choose a reason for hiding this comment

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

Say me a reason why do you not improve my changes?

Я специально вынес blockbody - чтобы за него цеплятся блоками, я специально удалил излишки h2 так как для сео они крайне неверны. для сео нужны конкретные вещи, а не то что вообще не должно индексироватся.... в общем откат крайне абсурден. Спасибо за понимание

@seditio
Copy link
Member Author

Choose a reason for hiding this comment

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

Абсолютно ненужные и излишние блоки в теме удалены за их ненужностью. Свое понимание SEO каждый реализует по-своему. Откат не более абсурден, чем избыточная разметка. У нас тут не WP или Джумла.

@esclkm
Copy link
Contributor

@esclkm esclkm commented on 69fbb2f Dec 22, 2011

Choose a reason for hiding this comment

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

категорически не согласен

  1. последний твой релиз был полгода назад - после этого ни одного принципиально комита
  2. твоя схема не позволяет сделать раскладку как на тренердиеты.рф - то есть заголовки отдельные от блоков.
  3. ну зачем сео роботу получать на каждой странице h2 - "теги" "администирирование" "комментарии", везде и всюду написано как большими буквами что заголовки должно быть h1 - один на страницу. h2 несколько на страницу h3 чуть больше. h1 - самое главное на странице, h2 мене важные части. у нас h1 - это заголовок сайта = title то есть h1 мы не используем вовсе. а в h2 на каждой странице "теги" "комментарии" - что опять убивает смысл данного тега.
  4. перед масс откатом можно было бы спросить. А не заявить что у нас не джумла. Крайне расстроен. спасибо

@seditio
Copy link
Member Author

Choose a reason for hiding this comment

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

для своих сайтиков добавишь такие блоки, какие хочешь. затачивать все под тебя как-то глупо, правда? остальное -- бред. впрочем, улучшения будут. кстати, я тоже был расстроен вымарыванием ссылки на seditio.by из плагина contact. и многими другими вещами. но как видишь -- ничего страшного.

@esclkm
Copy link
Contributor

@esclkm esclkm commented on 69fbb2f Dec 22, 2011

Choose a reason for hiding this comment

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

я вижу. добро пожаловать назад.
в принципе для своих сайтищев ты тоже можешь использховать кастом скин.

может еще админку откатишь????

@seditio
Copy link
Member Author

Choose a reason for hiding this comment

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

до админки дело еще дойдет. меня же никто не спрашивал прежде чем лезть и туда (как и в немезис). балаган предлагаю закрыть. извини за испорченный вечер.

@esclkm
Copy link
Contributor

@esclkm esclkm commented on 69fbb2f Dec 22, 2011

Choose a reason for hiding this comment

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

я в модулях хотябы могу вернуть лишний блок? еды он не просит а возможностей придает

@seditio
Copy link
Member Author

Choose a reason for hiding this comment

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

не надо. послушай, никаких возможностей он тебе не дает. для фантазий есть css и jquery.

@esclkm
Copy link
Contributor

@esclkm esclkm commented on 69fbb2f Dec 22, 2011

Choose a reason for hiding this comment

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

нет. поясню
структура блока такая:
< div class="block" >< h2 >ХЭДЕР< /h2 >КОНТЕНТ< /div >

Каким селектором выбрать КОНТЕНТ? я до сих не придумал - так как там у нас бывает сразу form ul p div или бовсе без селектора

@seditio
Copy link
Member Author

Choose a reason for hiding this comment

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

Без селектора -- это назад в LDU к Энтони.
Что касается четырех блочных элементов, опиши их в стилях как adjacent sibling -- каждый или любой.

@esclkm
Copy link
Contributor

@esclkm esclkm commented on 69fbb2f Dec 22, 2011

Choose a reason for hiding this comment

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

это мы с тобой когдато обсуждали - не получается.... особенно когда висит голый текст....
а делать .block * - задевает даже заголовок....

@seditio
Copy link
Member Author

Choose a reason for hiding this comment

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

еще раз: голый текст без разметки выводят только неправильные плагины (tags.index.php line 50).
не надо использовать избыточную разметку за счет нежелания добавить (и шаблонизировать) нужные блоки.
по поводу задевания заголовков: не переживай, верстальщик разберется.

@esclkm
Copy link
Contributor

@esclkm esclkm commented on 69fbb2f Dec 22, 2011

Choose a reason for hiding this comment

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

упрямо.... крайне упрямо... но без обосновано..эх...

@seditio
Copy link
Member Author

Choose a reason for hiding this comment

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

Не надо затачивать движок под себя, тем более такими безумными методами. Стиль и манеру общения тоже неплохо было бы изменить: 67672c3#commitcomment-714241
Что касается твоих бед, они решаются на уровне верстки сайта: http://www.seditio.by/blog/jquery/jquery-for-html-coding-and-markup-customization.html
Соавторство плагина contact я бы хотел получить назад -- объяснения его исчезновению я так и не услышал.

@esclkm
Copy link
Contributor

@esclkm esclkm commented on 69fbb2f Dec 23, 2011

Choose a reason for hiding this comment

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

клевое решение!
http://www.seditio.by/blog/jquery/jquery-for-html-coding-and-markup-customization.html

более универсального тяжело придумать! долго придумывал? у меня на 2 девайсах отключен - а значит описанный способ не работает. Да и концепция котонти что все должно симметрично работать с JS и без него
вопрос не для себя а о базовой возможности скина - который претендует на полную универсальность.

да контакт я планирую очен сильно изменить добавив под себя аякс. ) если еще нужны причины пиши в ЛС.

Да и заточка под себя : это когда убираешь возможность А. Добавляешь Б. которая необходима самому себе. Тут же А+Б = что равно расширение базового функционала

@seditio
Copy link
Member Author

Choose a reason for hiding this comment

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

Для своих девайсов используй tablet и mobile versions.
По универсальности: в скине не будет то, что излишне или надо удалить. Если тебе что-то нужно, добавляй в своем рабочем проекте, который использует nemesis, но не в базовой поставке. Иначе делай на WP -- это пример универсальности.

@esclkm
Copy link
Contributor

@esclkm esclkm commented on 69fbb2f Dec 24, 2011

Choose a reason for hiding this comment

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

спасибо за понимание

Буду ждать когда надоест. лить ревизии и востановлю WP если понимания не видно

Please sign in to comment.