TASK: Announcements #54
Conversation
* TASK: change soundcloud color * TASK: change H2 soundcloud color
| @@ -0,0 +1,6 @@ | |||
| 'Sfi.Kateheo:Announce': | |||
| </div> | ||
| ` | ||
| style = ${'<style> | ||
| .Media { |
There was a problem hiding this comment.
Эти стили буду сталкиваться с другими, с таким же именем. Просто поменяй имя класса.
| @@ -0,0 +1,133 @@ | |||
| prototype(Sfi.Kateheo:Announce) < prototype(Sfi.Kateheo:AbstractDocument) { | |||
| body.content = Neos.Neos:ContentComponent { | |||
There was a problem hiding this comment.
В этом файле шокирующее количество кода тупо продублирована из News.
Либо выдели повторяющиеся элементы в отдельные компоненты (лучше)
либо скопируй прототип если неохота возиться (как здесь https://github.com/psmb/KateheoDistr/blob/3b01a82651b2201946b19aa7e70bea9347afbd15/DistributionPackages/Sfi.Kateheo/Resources/Private/Fusion/NodeTypes/News/Article.fusion)
There was a problem hiding this comment.
тогда будут стили сталкиваться. Надо тогда в отдельные компоненты
There was a problem hiding this comment.
Вообще, насколько это критично, совпадение стилей? На текущий момент, вроде это и надо.
There was a problem hiding this comment.
ну если стили такие же будут, то они не будут сталкиваться. Если ты хочешь, чтобы они были разными, то тогда лучше делать отдельные компоненты, а так можно пойти по простому пути.
| @@ -42,13 +42,49 @@ prototype(Sfi.Kateheo:PageMain) < prototype(Sfi.Kateheo:AbstractDocument) { | |||
| collection = ${Search.query(site).nodeType("Sfi.Kateheo:News").sortDesc('date').from(0).limit(4).execute().toArray()} | |||
| } | |||
|
|
|||
| announceList = Flowpack.Listable:Collection { | |||
There was a problem hiding this comment.
Лучше выделить в отдельный компонент.
TASK: change H2 soundcloud color
No description provided.