fix(module:select) MaxTagCount behaves properly for non-responsive scenarios #1776

anddrzejb · 2021-07-26T13:05:14Z

🤔 This is a ...

🔗 Related issue link

💡 Background and solution

Fixed the bug. Added XML comments.
Fixed order in english docs.
Redid Chineese docs (missing translations).
Added test scenario for MaxTagCount when non-responsive

📝 Changelog

Language	Changelog
🇺🇸 English
🇨🇳 Chinese

☑️ 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

…enarios docs: added XML comments fixed order in English docs redid Chinese docs (missing translations) test: add scenario for MaxTagCount when non-responsive

github-actions · 2021-07-26T13:05:29Z

ElderJames · 2021-07-26T13:18:57Z

Hi @anddrzejb ,welcome back.

I am preparing to release 0.9, does this need to be merged in?

anddrzejb · 2021-07-26T13:29:50Z

@ElderJames if you want the fix to be in 0.9, then probably yes. But I see there are some issues that weren't there before. Probably came from merging with feature. I may not be able to fix it right away.

ElderJames · 2021-07-26T13:47:41Z

Ok, so let's release it in 0.9.1 this weekend.

anddrzejb · 2021-07-26T14:01:25Z

As you wish. I actually did manage to merge the changes in Select component coming from feature branch, but I am not in a hurry. The OP of #1746 may be in a hurry, but I do not know that 🤷🏻‍♂️ .

codecov · 2021-07-26T14:30:08Z

Codecov Report

Merging #1776 (42efa1f) into master (8b41465) will increase coverage by 0.15%.
The diff coverage is 44.44%.

❗ Current head 42efa1f differs from pull request most recent head 0eceefc. Consider uploading reports for the commit 0eceefc to get more accurate results

@@            Coverage Diff             @@
##           master    #1776      +/-   ##
==========================================
+ Coverage   20.78%   20.93%   +0.15%     
==========================================
  Files         478      478              
  Lines       30452    30462      +10     
==========================================
+ Hits         6328     6376      +48     
+ Misses      24124    24086      -38

Impacted Files	Coverage Δ
components/select/Select.razor	`79.48% <ø> (ø)`
components/select/internal/SelectContent.razor	`51.79% <ø> (+10.79%)`	⬆️
components/select/Select.razor.cs	`40.33% <38.14%> (+1.53%)`	⬆️
components/select/SelectOption.razor.cs	`77.39% <66.66%> (ø)`
components/select/internal/SelectContent.razor.cs	`49.03% <87.50%> (+4.33%)`	⬆️
components/tree/TreeNode.razor.cs	`0.00% <0.00%> (ø)`

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 8b41465...0eceefc. Read the comment docs.

…scenarios (ant-design-blazor#1776) * fix(module:select) MaxTagCount behaves property for non-responsive scenarios docs: added XML comments fixed order in English docs redid Chinese docs (missing translations) test: add scenario for MaxTagCount when non-responsive * merge with changes coming from feature branch * docs: example of MaxTagCount change Co-authored-by: James Yeung <shunjiey@hotmail.com>

* refactor(module: table): use EnumSelect for table filter (#1777) Co-authored-by: James Yeung <shunjiey@hotmail.com> * fix(module: tree): fix parent checkbox state calculation (#1781) * fix(module: select): remove ValueName null check for non-datasource approach (#1785) * fix(module: select): fix null value (#1786) Co-authored-by: James Yeung <shunjiey@hotmail.com> * fix(module: select): MaxTagCount behaves properly for non-responsive scenarios (#1776) * fix(module:select) MaxTagCount behaves property for non-responsive scenarios docs: added XML comments fixed order in English docs redid Chinese docs (missing translations) test: add scenario for MaxTagCount when non-responsive * merge with changes coming from feature branch * docs: example of MaxTagCount change Co-authored-by: James Yeung <shunjiey@hotmail.com> * refactor(interop): reorganization into classes and typescript tests (#1791) * fix(module:interop): reorganization into classes * comments clean-up * Add typescript test project * fix: sync with pull request #1765: support drag * github test action fix attempt * tests: jsinterop test mock fix * codecov added to ts tests * add generated split js to gitignore * fix: coverage github action Co-authored-by: James Yeung <shunjiey@hotmail.com> * fix(module: tree): one more fix to parent checkbox state calculation (#1781) (#1794) * fix(module: modal): error when confirm TaskCompletionSource SetResult repeat (#1795) Co-authored-by: James Yeung <shunjiey@hotmail.com> * fix(module: table): fix cascading table in column (#1732) * fix(module: table): fix cascading tables * Update Column.razor * Update Column,razor Co-authored-by: James Yeung <shunjiey@hotmail.com> * feat(module: tree): bring SearchExpression back (#1796) * fix(module: RangePicker): ShowTime issues addressed (#1788) * fix(module:rangePicker): year & month skip * fix(module:rangePicker): extra space removed on 2nd picker * fix(module:rangePicker): when ShowTime, click on OK moves to other picker * fix(module:rangePicker): adjust year skip button * fix(module:rangePicker): focus issue resolution * fix(module:rangePicker): remove incorrect value from input Co-authored-by: James Yeung <shunjiey@hotmail.com> * fix(module: overlay): fix overlay position when container element has border (#1797) * fix(module: overlay): use right positioning for bottom-right placement (#1799) * fix(module: date-picker): update resource files (#1800) Co-authored-by: James Yeung <shunjiey@hotmail.com> * fix(module: calendar): fix header select width (#1801) Co-authored-by: James Yeung <shunjiey@hotmail.com> * fix(module: layout): fix OnBreakpoint and OnCollapse only when they was changed (#1815) * fix(module: date-picker): fix ShowToday when ShowTime is true (#1819) Co-authored-by: Łukasz Błażewicz <lukasz.blazewicz@homebook.pl> * doc: fix api markdown for autocomplete (#1808) * Update index.en-US.md * Update index.zh-CN.md Co-authored-by: James Yeung <shunjiey@hotmail.com> * fix(module: table): missing sorter model on first change (#1823) * fix(module: upload): basic functionality restored (#1824) * fix(module: table): use "or" filter condition for List type built-in filter (#1804) Co-authored-by: James Yeung <shunjiey@hotmail.com> * fix(module: auto-complete): turn off the input `autocomplete` attribute (#1825) * fix(module: date-picker): remain millisecond value when change the picker value (#1829) * fix(module: input-number): `OnChange` cannot be triggered during keyboard input (#1830) * fix(module: table): missing `showTime` of built-in datetime filter (#1827) Co-authored-by: James Yeung <shunjiey@hotmail.com> * changelog 0.9.1 (#1831) Co-authored-by: anranruye <54608128+anranruye@users.noreply.github.com> Co-authored-by: lukblazewicz <39852149+lukblazewicz@users.noreply.github.com> Co-authored-by: Andrzej Bakun <andrzej@neelyc.com.cy> Co-authored-by: zxyao <zxyao145@gmail.com> Co-authored-by: Łukasz Błażewicz <lukasz.blazewicz@homebook.pl> Co-authored-by: heroboy <yangweiqin@gmail.com>

…scenarios (#1776) * fix(module:select) MaxTagCount behaves property for non-responsive scenarios docs: added XML comments fixed order in English docs redid Chinese docs (missing translations) test: add scenario for MaxTagCount when non-responsive * merge with changes coming from feature branch * docs: example of MaxTagCount change Co-authored-by: James Yeung <shunjiey@hotmail.com>

* refactor(module: table): use EnumSelect for table filter (#1777) Co-authored-by: James Yeung <shunjiey@hotmail.com> * fix(module: tree): fix parent checkbox state calculation (#1781) * fix(module: select): remove ValueName null check for non-datasource approach (#1785) * fix(module: select): fix null value (#1786) Co-authored-by: James Yeung <shunjiey@hotmail.com> * fix(module: select): MaxTagCount behaves properly for non-responsive scenarios (#1776) * fix(module:select) MaxTagCount behaves property for non-responsive scenarios docs: added XML comments fixed order in English docs redid Chinese docs (missing translations) test: add scenario for MaxTagCount when non-responsive * merge with changes coming from feature branch * docs: example of MaxTagCount change Co-authored-by: James Yeung <shunjiey@hotmail.com> * refactor(interop): reorganization into classes and typescript tests (#1791) * fix(module:interop): reorganization into classes * comments clean-up * Add typescript test project * fix: sync with pull request #1765: support drag * github test action fix attempt * tests: jsinterop test mock fix * codecov added to ts tests * add generated split js to gitignore * fix: coverage github action Co-authored-by: James Yeung <shunjiey@hotmail.com> * fix(module: tree): one more fix to parent checkbox state calculation (#1781) (#1794) * fix(module: modal): error when confirm TaskCompletionSource SetResult repeat (#1795) Co-authored-by: James Yeung <shunjiey@hotmail.com> * fix(module: table): fix cascading table in column (#1732) * fix(module: table): fix cascading tables * Update Column.razor * Update Column,razor Co-authored-by: James Yeung <shunjiey@hotmail.com> * feat(module: tree): bring SearchExpression back (#1796) * fix(module: RangePicker): ShowTime issues addressed (#1788) * fix(module:rangePicker): year & month skip * fix(module:rangePicker): extra space removed on 2nd picker * fix(module:rangePicker): when ShowTime, click on OK moves to other picker * fix(module:rangePicker): adjust year skip button * fix(module:rangePicker): focus issue resolution * fix(module:rangePicker): remove incorrect value from input Co-authored-by: James Yeung <shunjiey@hotmail.com> * fix(module: overlay): fix overlay position when container element has border (#1797) * fix(module: overlay): use right positioning for bottom-right placement (#1799) * fix(module: date-picker): update resource files (#1800) Co-authored-by: James Yeung <shunjiey@hotmail.com> * fix(module: calendar): fix header select width (#1801) Co-authored-by: James Yeung <shunjiey@hotmail.com> * fix(module: layout): fix OnBreakpoint and OnCollapse only when they was changed (#1815) * fix(module: date-picker): fix ShowToday when ShowTime is true (#1819) Co-authored-by: Łukasz Błażewicz <lukasz.blazewicz@homebook.pl> * doc: fix api markdown for autocomplete (#1808) * Update index.en-US.md * Update index.zh-CN.md Co-authored-by: James Yeung <shunjiey@hotmail.com> * fix(module: table): missing sorter model on first change (#1823) * fix(module: upload): basic functionality restored (#1824) * fix(module: table): use "or" filter condition for List type built-in filter (#1804) Co-authored-by: James Yeung <shunjiey@hotmail.com> * fix(module: auto-complete): turn off the input `autocomplete` attribute (#1825) * fix(module: date-picker): remain millisecond value when change the picker value (#1829) * fix(module: input-number): `OnChange` cannot be triggered during keyboard input (#1830) * fix(module: table): missing `showTime` of built-in datetime filter (#1827) Co-authored-by: James Yeung <shunjiey@hotmail.com> * changelog 0.9.1 (#1831) Co-authored-by: anranruye <54608128+anranruye@users.noreply.github.com> Co-authored-by: lukblazewicz <39852149+lukblazewicz@users.noreply.github.com> Co-authored-by: Andrzej Bakun <andrzej@neelyc.com.cy> Co-authored-by: zxyao <zxyao145@gmail.com> Co-authored-by: Łukasz Błażewicz <lukasz.blazewicz@homebook.pl> Co-authored-by: heroboy <yangweiqin@gmail.com>

…scenarios (#1776) * fix(module:select) MaxTagCount behaves property for non-responsive scenarios docs: added XML comments fixed order in English docs redid Chinese docs (missing translations) test: add scenario for MaxTagCount when non-responsive * merge with changes coming from feature branch * docs: example of MaxTagCount change Co-authored-by: James Yeung <shunjiey@hotmail.com>

* refactor(module: table): use EnumSelect for table filter (#1777) Co-authored-by: James Yeung <shunjiey@hotmail.com> * fix(module: tree): fix parent checkbox state calculation (#1781) * fix(module: select): remove ValueName null check for non-datasource approach (#1785) * fix(module: select): fix null value (#1786) Co-authored-by: James Yeung <shunjiey@hotmail.com> * fix(module: select): MaxTagCount behaves properly for non-responsive scenarios (#1776) * fix(module:select) MaxTagCount behaves property for non-responsive scenarios docs: added XML comments fixed order in English docs redid Chinese docs (missing translations) test: add scenario for MaxTagCount when non-responsive * merge with changes coming from feature branch * docs: example of MaxTagCount change Co-authored-by: James Yeung <shunjiey@hotmail.com> * refactor(interop): reorganization into classes and typescript tests (#1791) * fix(module:interop): reorganization into classes * comments clean-up * Add typescript test project * fix: sync with pull request #1765: support drag * github test action fix attempt * tests: jsinterop test mock fix * codecov added to ts tests * add generated split js to gitignore * fix: coverage github action Co-authored-by: James Yeung <shunjiey@hotmail.com> * fix(module: tree): one more fix to parent checkbox state calculation (#1781) (#1794) * fix(module: modal): error when confirm TaskCompletionSource SetResult repeat (#1795) Co-authored-by: James Yeung <shunjiey@hotmail.com> * fix(module: table): fix cascading table in column (#1732) * fix(module: table): fix cascading tables * Update Column.razor * Update Column,razor Co-authored-by: James Yeung <shunjiey@hotmail.com> * feat(module: tree): bring SearchExpression back (#1796) * fix(module: RangePicker): ShowTime issues addressed (#1788) * fix(module:rangePicker): year & month skip * fix(module:rangePicker): extra space removed on 2nd picker * fix(module:rangePicker): when ShowTime, click on OK moves to other picker * fix(module:rangePicker): adjust year skip button * fix(module:rangePicker): focus issue resolution * fix(module:rangePicker): remove incorrect value from input Co-authored-by: James Yeung <shunjiey@hotmail.com> * fix(module: overlay): fix overlay position when container element has border (#1797) * fix(module: overlay): use right positioning for bottom-right placement (#1799) * fix(module: date-picker): update resource files (#1800) Co-authored-by: James Yeung <shunjiey@hotmail.com> * fix(module: calendar): fix header select width (#1801) Co-authored-by: James Yeung <shunjiey@hotmail.com> * fix(module: layout): fix OnBreakpoint and OnCollapse only when they was changed (#1815) * fix(module: date-picker): fix ShowToday when ShowTime is true (#1819) Co-authored-by: Łukasz Błażewicz <lukasz.blazewicz@homebook.pl> * doc: fix api markdown for autocomplete (#1808) * Update index.en-US.md * Update index.zh-CN.md Co-authored-by: James Yeung <shunjiey@hotmail.com> * fix(module: table): missing sorter model on first change (#1823) * fix(module: upload): basic functionality restored (#1824) * fix(module: table): use "or" filter condition for List type built-in filter (#1804) Co-authored-by: James Yeung <shunjiey@hotmail.com> * fix(module: auto-complete): turn off the input `autocomplete` attribute (#1825) * fix(module: date-picker): remain millisecond value when change the picker value (#1829) * fix(module: input-number): `OnChange` cannot be triggered during keyboard input (#1830) * fix(module: table): missing `showTime` of built-in datetime filter (#1827) Co-authored-by: James Yeung <shunjiey@hotmail.com> * changelog 0.9.1 (#1831) Co-authored-by: anranruye <54608128+anranruye@users.noreply.github.com> Co-authored-by: lukblazewicz <39852149+lukblazewicz@users.noreply.github.com> Co-authored-by: Andrzej Bakun <andrzej@neelyc.com.cy> Co-authored-by: zxyao <zxyao145@gmail.com> Co-authored-by: Łukasz Błażewicz <lukasz.blazewicz@homebook.pl> Co-authored-by: heroboy <yangweiqin@gmail.com>

…scenarios (#1776) * fix(module:select) MaxTagCount behaves property for non-responsive scenarios docs: added XML comments fixed order in English docs redid Chinese docs (missing translations) test: add scenario for MaxTagCount when non-responsive * merge with changes coming from feature branch * docs: example of MaxTagCount change Co-authored-by: James Yeung <shunjiey@hotmail.com>

* refactor(module: table): use EnumSelect for table filter (#1777) Co-authored-by: James Yeung <shunjiey@hotmail.com> * fix(module: tree): fix parent checkbox state calculation (#1781) * fix(module: select): remove ValueName null check for non-datasource approach (#1785) * fix(module: select): fix null value (#1786) Co-authored-by: James Yeung <shunjiey@hotmail.com> * fix(module: select): MaxTagCount behaves properly for non-responsive scenarios (#1776) * fix(module:select) MaxTagCount behaves property for non-responsive scenarios docs: added XML comments fixed order in English docs redid Chinese docs (missing translations) test: add scenario for MaxTagCount when non-responsive * merge with changes coming from feature branch * docs: example of MaxTagCount change Co-authored-by: James Yeung <shunjiey@hotmail.com> * refactor(interop): reorganization into classes and typescript tests (#1791) * fix(module:interop): reorganization into classes * comments clean-up * Add typescript test project * fix: sync with pull request #1765: support drag * github test action fix attempt * tests: jsinterop test mock fix * codecov added to ts tests * add generated split js to gitignore * fix: coverage github action Co-authored-by: James Yeung <shunjiey@hotmail.com> * fix(module: tree): one more fix to parent checkbox state calculation (#1781) (#1794) * fix(module: modal): error when confirm TaskCompletionSource SetResult repeat (#1795) Co-authored-by: James Yeung <shunjiey@hotmail.com> * fix(module: table): fix cascading table in column (#1732) * fix(module: table): fix cascading tables * Update Column.razor * Update Column,razor Co-authored-by: James Yeung <shunjiey@hotmail.com> * feat(module: tree): bring SearchExpression back (#1796) * fix(module: RangePicker): ShowTime issues addressed (#1788) * fix(module:rangePicker): year & month skip * fix(module:rangePicker): extra space removed on 2nd picker * fix(module:rangePicker): when ShowTime, click on OK moves to other picker * fix(module:rangePicker): adjust year skip button * fix(module:rangePicker): focus issue resolution * fix(module:rangePicker): remove incorrect value from input Co-authored-by: James Yeung <shunjiey@hotmail.com> * fix(module: overlay): fix overlay position when container element has border (#1797) * fix(module: overlay): use right positioning for bottom-right placement (#1799) * fix(module: date-picker): update resource files (#1800) Co-authored-by: James Yeung <shunjiey@hotmail.com> * fix(module: calendar): fix header select width (#1801) Co-authored-by: James Yeung <shunjiey@hotmail.com> * fix(module: layout): fix OnBreakpoint and OnCollapse only when they was changed (#1815) * fix(module: date-picker): fix ShowToday when ShowTime is true (#1819) Co-authored-by: Łukasz Błażewicz <lukasz.blazewicz@homebook.pl> * doc: fix api markdown for autocomplete (#1808) * Update index.en-US.md * Update index.zh-CN.md Co-authored-by: James Yeung <shunjiey@hotmail.com> * fix(module: table): missing sorter model on first change (#1823) * fix(module: upload): basic functionality restored (#1824) * fix(module: table): use "or" filter condition for List type built-in filter (#1804) Co-authored-by: James Yeung <shunjiey@hotmail.com> * fix(module: auto-complete): turn off the input `autocomplete` attribute (#1825) * fix(module: date-picker): remain millisecond value when change the picker value (#1829) * fix(module: input-number): `OnChange` cannot be triggered during keyboard input (#1830) * fix(module: table): missing `showTime` of built-in datetime filter (#1827) Co-authored-by: James Yeung <shunjiey@hotmail.com> * changelog 0.9.1 (#1831) Co-authored-by: anranruye <54608128+anranruye@users.noreply.github.com> Co-authored-by: lukblazewicz <39852149+lukblazewicz@users.noreply.github.com> Co-authored-by: Andrzej Bakun <andrzej@neelyc.com.cy> Co-authored-by: zxyao <zxyao145@gmail.com> Co-authored-by: Łukasz Błażewicz <lukasz.blazewicz@homebook.pl> Co-authored-by: heroboy <yangweiqin@gmail.com>

fix(module:select) MaxTagCount behaves property for non-responsive sc…

4c68f7e

…enarios docs: added XML comments fixed order in English docs redid Chinese docs (missing translations) test: add scenario for MaxTagCount when non-responsive

pr-triage bot added the PR: unreviewed label Jul 26, 2021

Merge branch 'master' into selectTagMaxCount

46d0d24

anddrzejb mentioned this pull request Jul 26, 2021

MaxTagCount in Select on 'multiple' mode #1746

Closed

merge with changes coming from feature branch

5d2b93d

anddrzejb added 2 commits July 27, 2021 07:01

Merge branch 'master' into selectTagMaxCount

14292c5

docs: example of MaxTagCount change

42efa1f

anddrzejb changed the title ~~fix(module:select) MaxTagCount behaves property for non-responsive scenarios~~ fix(module:select) MaxTagCount behaves properly for non-responsive scenarios Jul 27, 2021

ElderJames approved these changes Jul 28, 2021

View reviewed changes

pr-triage bot added PR: reviewed-approved and removed PR: unreviewed labels Jul 28, 2021

Merge branch 'master' into selectTagMaxCount

0eceefc

pr-triage bot added PR: unreviewed and removed PR: reviewed-approved labels Jul 28, 2021

ElderJames enabled auto-merge (squash) July 28, 2021 14:19

ElderJames merged commit da5a3d0 into ant-design-blazor:master Jul 28, 2021

pr-triage bot added PR: merged and removed PR: unreviewed labels Jul 28, 2021

anddrzejb deleted the selectTagMaxCount branch July 29, 2021 12:11

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

fix(module:select) MaxTagCount behaves properly for non-responsive scenarios #1776

fix(module:select) MaxTagCount behaves properly for non-responsive scenarios #1776

anddrzejb commented Jul 26, 2021

github-actions bot commented Jul 26, 2021 •

edited

ElderJames commented Jul 26, 2021

anddrzejb commented Jul 26, 2021

ElderJames commented Jul 26, 2021

anddrzejb commented Jul 26, 2021

codecov bot commented Jul 26, 2021 •

edited

fix(module:select) MaxTagCount behaves properly for non-responsive scenarios #1776

fix(module:select) MaxTagCount behaves properly for non-responsive scenarios #1776

Conversation

anddrzejb commented Jul 26, 2021

🤔 This is a ...

🔗 Related issue link

💡 Background and solution

📝 Changelog

☑️ Self Check before Merge

github-actions bot commented Jul 26, 2021 • edited

ElderJames commented Jul 26, 2021

anddrzejb commented Jul 26, 2021

ElderJames commented Jul 26, 2021

anddrzejb commented Jul 26, 2021

codecov bot commented Jul 26, 2021 • edited

Codecov Report

github-actions bot commented Jul 26, 2021 •

edited

codecov bot commented Jul 26, 2021 •

edited