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

Комментарии слишком близко друг к другу #637

Closed
itcrowd72 opened this Issue Sep 18, 2017 · 10 comments

Comments

Projects
None yet
5 participants
@itcrowd72
Member

itcrowd72 commented Sep 18, 2017

bildschirmfoto vom 2017-09-18 12-14-54

Предлагаю в .main-content .node .comment значение margin-top увеличить до 15px;

Так же лучше:
bildschirmfoto vom 2017-09-18 12-16-41

@awd-studio

This comment has been minimized.

Show comment
Hide comment
@awd-studio

awd-studio Sep 18, 2017

Contributor

По-мне тоже немного слитно. + на телефонах будет удобнее.

Contributor

awd-studio commented Sep 18, 2017

По-мне тоже немного слитно. + на телефонах будет удобнее.

@hulygun

This comment has been minimized.

Show comment
Hide comment
@hulygun

hulygun Sep 18, 2017

Contributor

+ при новых комментах аватарка вплотную с полоской выглядит не айс

Contributor

hulygun commented Sep 18, 2017

+ при новых комментах аватарка вплотную с полоской выглядит не айс

@fairrandir

This comment has been minimized.

Show comment
Hide comment
@fairrandir

fairrandir Sep 19, 2017

плюс у многоуровневых комментариев уровень совсем не виден.
screenshot_1

fairrandir commented Sep 19, 2017

плюс у многоуровневых комментариев уровень совсем не виден.
screenshot_1

@mbaev

This comment has been minimized.

Show comment
Hide comment
@mbaev

mbaev Sep 19, 2017

Contributor

Тогда нужен другой дизайн.

Contributor

mbaev commented Sep 19, 2017

Тогда нужен другой дизайн.

@fairrandir

This comment has been minimized.

Show comment
Hide comment
@fairrandir

fairrandir Sep 19, 2017

Можно что-то типа такого:

.main-content .node .comment {
  background: #fff;
  margin: 0;
  padding: 10px;
  border-top: 1px solid #e6e6e6;
}

.indented {
  background: rgba(61, 118, 160, 0.15);
}

screenshot_2

fairrandir commented Sep 19, 2017

Можно что-то типа такого:

.main-content .node .comment {
  background: #fff;
  margin: 0;
  padding: 10px;
  border-top: 1px solid #e6e6e6;
}

.indented {
  background: rgba(61, 118, 160, 0.15);
}

screenshot_2

@itcrowd72

This comment has been minimized.

Show comment
Hide comment
@itcrowd72

itcrowd72 Sep 19, 2017

Member

border-top в тему, плюсую

Member

itcrowd72 commented Sep 19, 2017

border-top в тему, плюсую

@mbaev

This comment has been minimized.

Show comment
Hide comment
@mbaev

mbaev Sep 19, 2017

Contributor

@fairrandir очень классно!

Contributor

mbaev commented Sep 19, 2017

@fairrandir очень классно!

@mbaev mbaev self-assigned this Sep 19, 2017

@mbaev mbaev added this to the 2nd October 2017 milestone Sep 19, 2017

@mbaev mbaev added the Improvement label Sep 19, 2017

@mbaev mbaev modified the milestones: 2nd October 2017, 6th November 2017 Sep 23, 2017

@itcrowd72

This comment has been minimized.

Show comment
Hide comment
@itcrowd72

itcrowd72 Oct 11, 2017

Member

Закрываю, т.к. есть другие issue по этой теме #691 и #700

Member

itcrowd72 commented Oct 11, 2017

Закрываю, т.к. есть другие issue по этой теме #691 и #700

@itcrowd72 itcrowd72 closed this Oct 11, 2017

@mbaev

This comment has been minimized.

Show comment
Hide comment
@mbaev

mbaev Oct 11, 2017

Contributor

Нет, нет. Не закрывай свои тикеты, если нет решения.
Если это дубликат, то закроет кто-то другой.

Contributor

mbaev commented Oct 11, 2017

Нет, нет. Не закрывай свои тикеты, если нет решения.
Если это дубликат, то закроет кто-то другой.

@mbaev

This comment has been minimized.

Show comment
Hide comment
@mbaev

mbaev Oct 24, 2017

Contributor

Внесено улучшение по предложению от @fairrandir

image

Можно тестить на dev. Например, здесь: http://dev.drupal.ru/node/1#comment-287

Contributor

mbaev commented Oct 24, 2017

Внесено улучшение по предложению от @fairrandir

image

Можно тестить на dev. Например, здесь: http://dev.drupal.ru/node/1#comment-287

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment