Skip to content
This repository has been archived by the owner on Jan 5, 2024. It is now read-only.

[Feat] Scroll to top when Chart category changed #65

Closed
ChoiysApple opened this issue Dec 20, 2023 · 0 comments · Fixed by #67
Closed

[Feat] Scroll to top when Chart category changed #65

ChoiysApple opened this issue Dec 20, 2023 · 0 comments · Fixed by #67
Labels
✨ Feature New Feature
Milestone
0.1.1

Comments

@ChoiysApple
Copy link
Owner

Description

Current Chart list is not changing scroll location

Benefit

Better User Experience. (User need to see list from first when they change category)

Additional context

Current version has diffrent action by number of rows

  • numberOfRows <= 20 Scroll location not changes
  • numberOfRows > 20 Scrolled to first row but not top (Navigation title is not presented as large title)
@ChoiysApple ChoiysApple added the ✨ Feature New Feature label Dec 20, 2023
@ChoiysApple ChoiysApple added this to the 0.1.1 milestone Dec 20, 2023
This was referenced Jan 1, 2024
Open
Merged
ChoiysApple added a commit that referenced this issue Jan 2, 2024
@ChoiysApple
[Feature] #65 Scroll to top when Chart category changed (#67)
ce7515a 
<!--- 
Pull Request title must contains Category, issue number, issue
description
(e.g. "[Feature] #123 Adding new Gestures")
-->

### Describe
New feature when changing category 

### Changes Made
Scroll automatically go to top when category changed


### Issues Resolved
- #65 
<!-- 
If this pull request addresses or closes any related issues, mention
them here. Use the GitHub issue linking format
(e.g., "Feature #123"). 
-->

### Additional context
Issue related to Refresh control is found & created:
- #66
@ChoiysApple ChoiysApple linked a pull request Jan 2, 2024 that will close this issue
[Feature] #65 Scroll to top when Chart category changed #67
Merged
@ChoiysApple ChoiysApple mentioned this issue Jan 3, 2024
Merged
ChoiysApple added a commit that referenced this issue Jan 3, 2024
@ChoiysApple
[Realese] version 0.1.1 (#68)
c6ef7ba 
End of milestone 0.1.1

### Issue Resolved
- #57 
- #58
- #63 
- #65
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees
No one assigned
Labels
✨ Feature New Feature
Projects
No open projects
Filmboard Refactoring
Status: Deployed
Milestone
0.1.1
Development

Successfully merging a pull request may close this issue.

[Feature] #65 Scroll to top when Chart category changed ChoiysApple/Filmboard
1 participant
@ChoiysApple