Skip to content
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

fix(module:rate): Value has priority over DefaultValue #1303

Merged
merged 9 commits into from
Apr 2, 2021

Conversation

anddrzejb
Copy link
Member

🤔 This is a ...

  • New feature
  • Bug fix
  • Site / documentation update
  • Demo update
  • Component style update
  • Bundle size optimization
  • Performance optimization
  • Refactoring
  • Code style optimization
  • Test Case
  • Branch merge
  • Other (about what?)

🔗 Related issue link

If both Value and DefaultValue is set, the Value is ignored. In the blow code, rate shows value "2" instead of "3".

<Rate @bind-Value=@value DefaultValue="2"/>

@code {
    decimal value = 3;    
}

💡 Background and solution

Introduced value changed check. If value was set, then default value is ignored.

☑️ Self Check before Merge

⚠️ Please check all items below before review. ⚠️

  • Doc is updated/provided or not needed
  • Demo is updated/provided or not needed
  • Changelog is provided or not needed

@github-actions
Copy link

github-actions bot commented Apr 2, 2021

@ElderJames ElderJames merged commit 5f14377 into ant-design-blazor:master Apr 2, 2021
@anddrzejb anddrzejb deleted the rateDefaultFix branch April 2, 2021 14:54
ElderJames added a commit that referenced this pull request Apr 8, 2021
* fix(module: table): ExpandableRow not working with ActionColumn #1284 (#1285)

* (fix) Table ExpandableRow not working with ActionColumn #1284

* Add Tree Button Expand on Action Columm.

Check the position of Selection Column when calculate the position for expand button on tree mode and expand mode.

* fix(module: rate): Value has priority over DefaultValue (#1303)

* chore: add GitHub Actions for auto preview (#1205)

* chore: add github actions for auto preview

* fix setup job

* chore: add GitHub Actions for auto preview (#1205)

* fix(module:rate): prevent reset to default value

Co-authored-by: James Yeung <shunjiey@hotmail.com>

* feat(module: form): Add LabelTemplate for FormItem (#1293)

Co-authored-by: Leishi <lluo@octet.com>

* fix(module: table): fix DataIndex Column with incorrect sorting and filter (#1295)

* refactor(module: table): Unify the behavior of Field and DataIndex

* fix(module: table): Fix DataIndex Column doesn't refresh

* doc(module: table): add DataIndex column filter demo

* doc(module: table): fix Blazor table

Co-authored-by: James Yeung <shunjiey@hotmail.com>

* fix(module: datepicker): DatePicker DisabledDate works incorect (#1298)

* fix(module: datepicker): DatePicker DisabledDate works incorect

* test(module: dateHelperTests): modify test case

Co-authored-by: James Yeung <shunjiey@hotmail.com>

* fix(module: upload): Upload list update fix (#1301)

* chore: add GitHub Actions for auto preview (#1205)

* chore: add github actions for auto preview

* fix setup job

* chore: add GitHub Actions for auto preview (#1205)

* fix(module:upload): file list update

* fix(module:upload): typed args for file upload

* fix(module:upload): delete button remove from picture-card

Co-authored-by: James Yeung <shunjiey@hotmail.com>

* fix(module: textarea): default to empty string (#1305)

* chore: add GitHub Actions for auto preview (#1205)

* chore: add github actions for auto preview

* fix setup job

* chore: add GitHub Actions for auto preview (#1205)

* fix(module:textarea): add parameter DefaultToEmptyString

* fix(module:textarea): occasional crashes of js in firefox

* docs(module:input): added new param DefaultToEmptyString

Co-authored-by: James Yeung <shunjiey@hotmail.com>

* fix(module: grid): missing flex and wrap style (#1296)

* fix(module: grid): missing flex style

* remove the old md file

* fix demos

* fix build error

* fix no warp class

* docs: add changelog in document (#1306)

* fix(module: select): load icon for multiple mode & value reload on delay (#1307)

* chore: add GitHub Actions for auto preview (#1205)

* chore: add github actions for auto preview

* fix setup job

* chore: add GitHub Actions for auto preview (#1205)

* fix(module:select): add loading for multiple mode

* fix(module:select): selected items duplication for multiple mode when values changed outside the component

Co-authored-by: James Yeung <shunjiey@hotmail.com>

* fix(module: table): perf optimization & data source change issue (#1304)

* chore: add GitHub Actions for auto preview (#1205)

* chore: add github actions for auto preview

* fix setup job

* chore: add GitHub Actions for auto preview (#1205)

* fix(module:table): guard against false entries in _dataSourceCache

* fix(module:table): perf - double to single render on datasource change

* fix(module:table): perf optimization to avoid multiple renders

Co-authored-by: James Yeung <shunjiey@hotmail.com>

* fix(module: select): propagation on item remove (#1308)

* fix(module: table): fix table re-render when ScrollX is set (#1311)

* docs(module: table): fix RouterPagging demo (#1313)

* chagnelog 0.7.4 (#1321)

Co-authored-by: Magehernan <magehernan@gmail.com>
Co-authored-by: Andrzej Bakun <andrzej@neelyc.com.cy>
Co-authored-by: ldsenow <ldsenow@gmail.com>
Co-authored-by: Leishi <lluo@octet.com>
Co-authored-by: Zonciu Liang <zonciu@zonciu.com>
Co-authored-by: 笨木头 <musicvs@163.com>
ElderJames added a commit that referenced this pull request Apr 23, 2022
* chore: add GitHub Actions for auto preview (#1205)

* chore: add github actions for auto preview

* fix setup job

* chore: add GitHub Actions for auto preview (#1205)

* fix(module:rate): prevent reset to default value

Co-authored-by: James Yeung <shunjiey@hotmail.com>
ElderJames added a commit that referenced this pull request Apr 23, 2022
* fix(module: table): ExpandableRow not working with ActionColumn #1284 (#1285)

* (fix) Table ExpandableRow not working with ActionColumn #1284

* Add Tree Button Expand on Action Columm.

Check the position of Selection Column when calculate the position for expand button on tree mode and expand mode.

* fix(module: rate): Value has priority over DefaultValue (#1303)

* chore: add GitHub Actions for auto preview (#1205)

* chore: add github actions for auto preview

* fix setup job

* chore: add GitHub Actions for auto preview (#1205)

* fix(module:rate): prevent reset to default value

Co-authored-by: James Yeung <shunjiey@hotmail.com>

* feat(module: form): Add LabelTemplate for FormItem (#1293)

Co-authored-by: Leishi <lluo@octet.com>

* fix(module: table): fix DataIndex Column with incorrect sorting and filter (#1295)

* refactor(module: table): Unify the behavior of Field and DataIndex

* fix(module: table): Fix DataIndex Column doesn't refresh

* doc(module: table): add DataIndex column filter demo

* doc(module: table): fix Blazor table

Co-authored-by: James Yeung <shunjiey@hotmail.com>

* fix(module: datepicker): DatePicker DisabledDate works incorect (#1298)

* fix(module: datepicker): DatePicker DisabledDate works incorect

* test(module: dateHelperTests): modify test case

Co-authored-by: James Yeung <shunjiey@hotmail.com>

* fix(module: upload): Upload list update fix (#1301)

* chore: add GitHub Actions for auto preview (#1205)

* chore: add github actions for auto preview

* fix setup job

* chore: add GitHub Actions for auto preview (#1205)

* fix(module:upload): file list update

* fix(module:upload): typed args for file upload

* fix(module:upload): delete button remove from picture-card

Co-authored-by: James Yeung <shunjiey@hotmail.com>

* fix(module: textarea): default to empty string (#1305)

* chore: add GitHub Actions for auto preview (#1205)

* chore: add github actions for auto preview

* fix setup job

* chore: add GitHub Actions for auto preview (#1205)

* fix(module:textarea): add parameter DefaultToEmptyString

* fix(module:textarea): occasional crashes of js in firefox

* docs(module:input): added new param DefaultToEmptyString

Co-authored-by: James Yeung <shunjiey@hotmail.com>

* fix(module: grid): missing flex and wrap style (#1296)

* fix(module: grid): missing flex style

* remove the old md file

* fix demos

* fix build error

* fix no warp class

* docs: add changelog in document (#1306)

* fix(module: select): load icon for multiple mode & value reload on delay (#1307)

* chore: add GitHub Actions for auto preview (#1205)

* chore: add github actions for auto preview

* fix setup job

* chore: add GitHub Actions for auto preview (#1205)

* fix(module:select): add loading for multiple mode

* fix(module:select): selected items duplication for multiple mode when values changed outside the component

Co-authored-by: James Yeung <shunjiey@hotmail.com>

* fix(module: table): perf optimization & data source change issue (#1304)

* chore: add GitHub Actions for auto preview (#1205)

* chore: add github actions for auto preview

* fix setup job

* chore: add GitHub Actions for auto preview (#1205)

* fix(module:table): guard against false entries in _dataSourceCache

* fix(module:table): perf - double to single render on datasource change

* fix(module:table): perf optimization to avoid multiple renders

Co-authored-by: James Yeung <shunjiey@hotmail.com>

* fix(module: select): propagation on item remove (#1308)

* fix(module: table): fix table re-render when ScrollX is set (#1311)

* docs(module: table): fix RouterPagging demo (#1313)

* chagnelog 0.7.4 (#1321)

Co-authored-by: Magehernan <magehernan@gmail.com>
Co-authored-by: Andrzej Bakun <andrzej@neelyc.com.cy>
Co-authored-by: ldsenow <ldsenow@gmail.com>
Co-authored-by: Leishi <lluo@octet.com>
Co-authored-by: Zonciu Liang <zonciu@zonciu.com>
Co-authored-by: 笨木头 <musicvs@163.com>
ElderJames added a commit that referenced this pull request Apr 30, 2022
* chore: add GitHub Actions for auto preview (#1205)

* chore: add github actions for auto preview

* fix setup job

* chore: add GitHub Actions for auto preview (#1205)

* fix(module:rate): prevent reset to default value

Co-authored-by: James Yeung <shunjiey@hotmail.com>
ElderJames added a commit that referenced this pull request Apr 30, 2022
* fix(module: table): ExpandableRow not working with ActionColumn #1284 (#1285)

* (fix) Table ExpandableRow not working with ActionColumn #1284

* Add Tree Button Expand on Action Columm.

Check the position of Selection Column when calculate the position for expand button on tree mode and expand mode.

* fix(module: rate): Value has priority over DefaultValue (#1303)

* chore: add GitHub Actions for auto preview (#1205)

* chore: add github actions for auto preview

* fix setup job

* chore: add GitHub Actions for auto preview (#1205)

* fix(module:rate): prevent reset to default value

Co-authored-by: James Yeung <shunjiey@hotmail.com>

* feat(module: form): Add LabelTemplate for FormItem (#1293)

Co-authored-by: Leishi <lluo@octet.com>

* fix(module: table): fix DataIndex Column with incorrect sorting and filter (#1295)

* refactor(module: table): Unify the behavior of Field and DataIndex

* fix(module: table): Fix DataIndex Column doesn't refresh

* doc(module: table): add DataIndex column filter demo

* doc(module: table): fix Blazor table

Co-authored-by: James Yeung <shunjiey@hotmail.com>

* fix(module: datepicker): DatePicker DisabledDate works incorect (#1298)

* fix(module: datepicker): DatePicker DisabledDate works incorect

* test(module: dateHelperTests): modify test case

Co-authored-by: James Yeung <shunjiey@hotmail.com>

* fix(module: upload): Upload list update fix (#1301)

* chore: add GitHub Actions for auto preview (#1205)

* chore: add github actions for auto preview

* fix setup job

* chore: add GitHub Actions for auto preview (#1205)

* fix(module:upload): file list update

* fix(module:upload): typed args for file upload

* fix(module:upload): delete button remove from picture-card

Co-authored-by: James Yeung <shunjiey@hotmail.com>

* fix(module: textarea): default to empty string (#1305)

* chore: add GitHub Actions for auto preview (#1205)

* chore: add github actions for auto preview

* fix setup job

* chore: add GitHub Actions for auto preview (#1205)

* fix(module:textarea): add parameter DefaultToEmptyString

* fix(module:textarea): occasional crashes of js in firefox

* docs(module:input): added new param DefaultToEmptyString

Co-authored-by: James Yeung <shunjiey@hotmail.com>

* fix(module: grid): missing flex and wrap style (#1296)

* fix(module: grid): missing flex style

* remove the old md file

* fix demos

* fix build error

* fix no warp class

* docs: add changelog in document (#1306)

* fix(module: select): load icon for multiple mode & value reload on delay (#1307)

* chore: add GitHub Actions for auto preview (#1205)

* chore: add github actions for auto preview

* fix setup job

* chore: add GitHub Actions for auto preview (#1205)

* fix(module:select): add loading for multiple mode

* fix(module:select): selected items duplication for multiple mode when values changed outside the component

Co-authored-by: James Yeung <shunjiey@hotmail.com>

* fix(module: table): perf optimization & data source change issue (#1304)

* chore: add GitHub Actions for auto preview (#1205)

* chore: add github actions for auto preview

* fix setup job

* chore: add GitHub Actions for auto preview (#1205)

* fix(module:table): guard against false entries in _dataSourceCache

* fix(module:table): perf - double to single render on datasource change

* fix(module:table): perf optimization to avoid multiple renders

Co-authored-by: James Yeung <shunjiey@hotmail.com>

* fix(module: select): propagation on item remove (#1308)

* fix(module: table): fix table re-render when ScrollX is set (#1311)

* docs(module: table): fix RouterPagging demo (#1313)

* chagnelog 0.7.4 (#1321)

Co-authored-by: Magehernan <magehernan@gmail.com>
Co-authored-by: Andrzej Bakun <andrzej@neelyc.com.cy>
Co-authored-by: ldsenow <ldsenow@gmail.com>
Co-authored-by: Leishi <lluo@octet.com>
Co-authored-by: Zonciu Liang <zonciu@zonciu.com>
Co-authored-by: 笨木头 <musicvs@163.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants