Robert Korulczyk

Results 275 comments of Robert Korulczyk

@Eseperio Navbar is directly connected with Twitter Bootstrap - without this class it would not work as expected.

@Eseperio Technically framework core is not related to any CSS framework. It uses defaults compatible with Bootstrap for practical reasons, but it does not mean that it is "BootstrapLinkPager". If...

I would prefer some helper which will provide API similar to defining form fields: ```php Grid::column('full_name')->asHtml()->label('Name')->value(function(Order $model) { return Html::tag('span', $model->first_name . '' . $model->last_name); }), ```

> The property \yii\grid\Column::grid is mandatory for the columns to render The idea is to generate array or entity with column configuration. You don't need to create `Column` instance here,...

I'm already using helpers for generating grid columns in Yii 1.1 app. It's like ```php GridColumn::boolean('active'), ``` vs ```php [ 'name' => 'active', 'type' => 'boolean', 'filter' => [ 1...

@cebe Queries with big offset can be really slow on big sets of data. Setting limit in this way can save lots of server resources.

I'm for ```php $this->createTable('tableName', [ 'id' => $this->primaryKey(), 'name' => $this->string()->index('indexName'), 'topic_id' => $this->integer()->foreignKey('indexName', 'referencedTable', 'referencedColumn'), ]); ```

We already have `createIndex()` for complex indexes.

> I'm in favor for chaining like @kjusupov exampled. This fits with the current way of constructing the migrations. This syntax does not give you any real benefits, it creates...

> A better approach I think would be to make it mandatory to include the keys in the array, e.g: Now you need to remember keys names, and IDE will...