-
Notifications
You must be signed in to change notification settings - Fork 12
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
Fixed css bugs and added features to datagrid #597
Conversation
8dbb713
to
7ac9d06
Compare
7ac9d06
to
770282c
Compare
@@ -442,7 +444,7 @@ define(function() { | |||
if (!!this.options.selectItem && !!this.options.selectItem.type) { | |||
var $cell = this.sandbox.dom.createElement(templates.headerCell); | |||
this.sandbox.dom.addClass($cell, constants.checkboxCellClass); | |||
if (this.options.selectItem.type === selectItems.CHECKBOX) { | |||
if (this.options.selectItem.type === selectItems.CHECKBOX && this.options.disabledItems.length === 0) { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
What is rendered if there are some disabledItems
?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Or the better question is: Shouldn't there be a possibility to select all non-disabled items?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
i have introduced this feature but i don't need it in the master PR i will extract it into an own PR which can be discussed separately.
@wachterjohannes We should also create better PRs in Husky, a simple link to the main PR is not enough. What exact css bugs are fixed here? That's probably not described anywhere, or is it? |
ace2d82
to
6cc47fd
Compare
* BUGFIX #616 Fixed label tick in firefox | ||
* ENHANCEMENT #615 Introduced no-img-icon as function to use record dependent icons | ||
* BUGFIX #613 Fixed search-icon to prevent routing | ||
* FEATURE #610 Added save parameter for datagrid | ||
======= | ||
* FEATURE #597 Fixed css bugs and added features to datagrid | ||
>>>>>>> c820cfb... added disabled items in table view |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please fix this conflict.
6cc47fd
to
98e7a38
Compare
98e7a38
to
34f415a
Compare
Fixed css bugs and added features to datagrid
Main PR sulu/sulu#1927