ember-basic-dropdown icon indicating copy to clipboard operation
ember-basic-dropdown copied to clipboard

use default values if a property (that can come from an argument) has a value that is `undefined`

Open emattias opened this issue 5 years ago • 4 comments
trafficstars

This makes it so that the default values are used even if arguments are sent in but if their values are undefined the defaults will be used.

For example ember-power-select always sends some arguments to BasicDropdown which caused the default values to not be used.

This should fix that.

(This pr is done for v2 branch since we cant upgrade to v3 yet because we use https://github.com/cibernox/ember-power-select-with-create which requires ember-basic-dropdown v2)

emattias avatar May 22 '20 13:05 emattias

Just to check, did you try v3? I think that in V3 it's solved.

Not that I can't merge it in v2, tho.

cibernox avatar May 22 '20 13:05 cibernox

Just to check, did you try v3? I think that in V3 it's solved.

Not that I can't merge it in v2, tho.

I added this while you wrote your comment :) ->

This pr is done for v2 branch since we cant upgrade to v3 yet because we use https://github.com/cibernox/ember-power-select-with-create which requires ember-basic-dropdown v2

emattias avatar May 22 '20 13:05 emattias

@emattias thanks for pointing that up. I've update ember-power-select-with-create to EPS 4.0 and ember-basic-dropdown 3.0, so you should be able to update now.

If you can, I'd rather leave v2 alone.

cibernox avatar May 22 '20 15:05 cibernox

@emattias thanks for pointing that up. I've update ember-power-select-with-create to EPS 4.0 and ember-basic-dropdown 3.0, so you should be able to update now.

If you can, I'd rather leave v2 alone.

Thanks but we cant update to 3.0 right now unfortunately. Either we use the fork or if you can, please merge these changes.

emattias avatar Jun 01 '20 15:06 emattias