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鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

refactor(module: tabs): improve rendering #1970

Merged
merged 14 commits into from
Oct 8, 2021
Merged

refactor(module: tabs): improve rendering #1970

merged 14 commits into from
Oct 8, 2021

Conversation

ElderJames
Copy link
Member

@ElderJames ElderJames commented Sep 24, 2021

馃 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

馃挕 Background and solution

  1. Reduced object allocation - Previously, a loop was used to render tabs and tabpane in order to control the order of rendering in tabs, but this resulted in a Cartesian product of objects growing in number. Now we render the tabpane in ChildContnet directly, replacing only the tab's title when in drag and drop.
  2. Fixed the scrolling position.

馃摑 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

@github-actions
Copy link

github-actions bot commented Sep 24, 2021

@codecov
Copy link

codecov bot commented Sep 27, 2021

Codecov Report

Merging #1970 (9a96d89) into master (1f2a7f6) will increase coverage by 1.90%.
The diff coverage is 67.82%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #1970      +/-   ##
==========================================
+ Coverage   25.39%   27.29%   +1.90%     
==========================================
  Files         488      511      +23     
  Lines       32019    24485    -7534     
  Branches        0      233     +233     
==========================================
- Hits         8130     6683    -1447     
+ Misses      23889    17766    -6123     
- Partials        0       36      +36     
Impacted Files Coverage 螖
components/tabs/TabPane.razor.cs 68.57% <66.07%> (-26.75%) 猬囷笍
components/tabs/Tabs.razor.cs 65.07% <67.03%> (+4.22%) 猬嗭笍
components/tabs/Tabs.razor 77.77% <77.77%> (+6.34%) 猬嗭笍
components/tabs/TabPane.razor 86.66% <81.81%> (-6.20%) 猬囷笍
components/core/Helpers/MemberPath/PathNode.cs 46.15% <0.00%> (-8.85%) 猬囷笍
components/core/Reflection/TypeDefined.cs 77.77% <0.00%> (-7.94%) 猬囷笍
components/input/InputGroup.razor.cs 87.50% <0.00%> (-6.95%) 猬囷笍
...ponents/date-picker/types/DatePickerPlaceholder.cs 22.72% <0.00%> (-4.20%) 猬囷笍
... and 424 more

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 1f2a7f6...9a96d89. Read the comment docs.

@ElderJames ElderJames merged commit 6459e80 into master Oct 8, 2021
@ElderJames ElderJames deleted the refactor/tabs branch October 8, 2021 06:25
ElderJames added a commit that referenced this pull request Oct 16, 2021
* refactor(module: tabs): improve rendering

* fix ForceRender & change constants to enums

* fix animated

* fix left extra content

* add Centered feature

* fix scroll distance

* fix scroll gap

* fix index out of range

* fix tests

* fix scroll distance

* fix dropdown button position

* fix ping left & right

* hide the dropdown button
ElderJames added a commit that referenced this pull request Apr 23, 2022
* refactor(module: tabs): improve rendering

* fix ForceRender & change constants to enums

* fix animated

* fix left extra content

* add Centered feature

* fix scroll distance

* fix scroll gap

* fix index out of range

* fix tests

* fix scroll distance

* fix dropdown button position

* fix ping left & right

* hide the dropdown button
ElderJames added a commit that referenced this pull request Apr 30, 2022
* refactor(module: tabs): improve rendering

* fix ForceRender & change constants to enums

* fix animated

* fix left extra content

* add Centered feature

* fix scroll distance

* fix scroll gap

* fix index out of range

* fix tests

* fix scroll distance

* fix dropdown button position

* fix ping left & right

* hide the dropdown button
ElderJames added a commit that referenced this pull request Sep 6, 2022
* refactor(module: tabs): improve rendering

* fix ForceRender & change constants to enums

* fix animated

* fix left extra content

* add Centered feature

* fix scroll distance

* fix scroll gap

* fix index out of range

* fix tests

* fix scroll distance

* fix dropdown button position

* fix ping left & right

* hide the dropdown button
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