Skip to content

TASK: Announcements#54

Open
zaveryukha wants to merge 32 commits intomasterfrom
anounce
Open

TASK: Announcements#54
zaveryukha wants to merge 32 commits intomasterfrom
anounce

Conversation

@zaveryukha
Copy link
Copy Markdown
Member

No description provided.

@@ -0,0 +1,6 @@
'Sfi.Kateheo:Announce':
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Announcement на английском

</div>
`
style = ${'<style>
.Media {
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Эти стили буду сталкиваться с другими, с таким же именем. Просто поменяй имя класса.

@@ -0,0 +1,133 @@
prototype(Sfi.Kateheo:Announce) < prototype(Sfi.Kateheo:AbstractDocument) {
body.content = Neos.Neos:ContentComponent {
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

В этом файле шокирующее количество кода тупо продублирована из News.
Либо выдели повторяющиеся элементы в отдельные компоненты (лучше)
либо скопируй прототип если неохота возиться (как здесь https://github.com/psmb/KateheoDistr/blob/3b01a82651b2201946b19aa7e70bea9347afbd15/DistributionPackages/Sfi.Kateheo/Resources/Private/Fusion/NodeTypes/News/Article.fusion)

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

тогда будут стили сталкиваться. Надо тогда в отдельные компоненты

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

Вообще, насколько это критично, совпадение стилей? На текущий момент, вроде это и надо.

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

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

collection = ${Search.query(site).nodeType("Sfi.Kateheo:News").sortDesc('date').from(0).limit(4).execute().toArray()}
}

announceList = Flowpack.Listable:Collection {
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Лучше выделить в отдельный компонент.

@zaveryukha zaveryukha changed the title TASK: Announcements FEATURE: Announcements May 1, 2019
@zaveryukha zaveryukha changed the title FEATURE: Announcements TASK: Announcements Dec 27, 2020
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.

2 participants