Skip to content

Conversation

@nataschachu
Copy link
Contributor

No description provided.

Copy link
Member

Choose a reason for hiding this comment

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

В какой библиотеке лежит блок image?

@mishaberezin
Copy link
Member

Блок image лежит в islands-components, здесь его использовать нельзя.

@nataschachu
Copy link
Contributor Author

Ребята, а давайте все-таки как-то сделаем более универсальное решение. Чтобы не только b-icon искались, но и в просто хотя бы img? Или элементы слайдера 'image'?

@mishaberezin
Copy link
Member

Можешь завести задачку в ISL с описанием проблемы?

@andre487
Copy link
Contributor

andre487 commented Apr 3, 2015

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

Нужно проводить как можно более детальную декомпозицию, и можно даже в таких вот местах блоки выносить в конфиги. Как-то так круто было бы сделать:

_getImages: function() {
    return this.findBlocksInside('b-icon');
}

Чтобы на проекте можно было переопределить метод.

Или как-то так:

_images: ['b-icon'],
// ...
_getImages: function() {
    return this._images.map(this._getImage, this);
},

_getImage: function(name) {
    return this.findBlocksInside(name);
}

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

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

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants