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

feat(ui): finish setting page #21

Merged
merged 10 commits into from
Aug 22, 2024
Merged

feat(ui): finish setting page #21

merged 10 commits into from
Aug 22, 2024

Conversation

DTPA101
Copy link
Collaborator

@DTPA101 DTPA101 commented Aug 13, 2024

No description provided.

@Serein207 Serein207 linked an issue Aug 13, 2024 that may be closed by this pull request
@cEvolve05 cEvolve05 added the feature New feature or request label Aug 13, 2024
Copy link

clang-tidy review says "All clean, LGTM! 👍"

Copy link

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

clang-tidy made some suggestions

src/Controller/View/SettingPage.cc Outdated Show resolved Hide resolved
src/Controller/View/SettingPage.cc Outdated Show resolved Hide resolved
Copy link

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

clang-tidy made some suggestions

src/Controller/View/SettingPage.cc Outdated Show resolved Hide resolved
src/Controller/View/SettingPage.cc Outdated Show resolved Hide resolved
Copy link

clang-tidy review says "All clean, LGTM! 👍"

Copy link

clang-tidy review says "All clean, LGTM! 👍"

@Serein207 Serein207 marked this pull request as ready for review August 22, 2024 12:09
Copy link

clang-tidy review says "All clean, LGTM! 👍"

Copy link

clang-tidy review says "All clean, LGTM! 👍"

@Serein207 Serein207 merged commit c772f8c into dev Aug 22, 2024
5 checks passed
@Serein207 Serein207 deleted the feat/ui/setting-page branch August 22, 2024 12:41
Serein207 added a commit that referenced this pull request Aug 22, 2024
* feat:some basic laying

* feat(ui): basic laying on the page

* ui laying primary draft

* feat(ui): callbacks and root

* attempt to fill in .cc and .slint

* callbacks fiied

* feat: setting page

* feat: change theme when app start

* chore: clean up code

* feat(ui): add scroll view in setting page

---------

Co-authored-by: Serein <[email protected]>
Serein207 added a commit that referenced this pull request Aug 22, 2024
* feat:some basic laying

* feat(ui): basic laying on the page

* ui laying primary draft

* feat(ui): callbacks and root

* attempt to fill in .cc and .slint

* callbacks fiied

* feat: setting page

* feat: change theme when app start

* chore: clean up code

* feat(ui): add scroll view in setting page

---------

Co-authored-by: Serein <[email protected]>
Serein207 added a commit that referenced this pull request Aug 22, 2024
* feat:some basic laying

* feat(ui): basic laying on the page

* ui laying primary draft

* feat(ui): callbacks and root

* attempt to fill in .cc and .slint

* callbacks fiied

* feat: setting page

* feat: change theme when app start

* chore: clean up code

* feat(ui): add scroll view in setting page

---------

Co-authored-by: Serein <[email protected]>
aurora0x27 pushed a commit that referenced this pull request Aug 29, 2024
* feat:some basic laying

* feat(ui): basic laying on the page

* ui laying primary draft

* feat(ui): callbacks and root

* attempt to fill in .cc and .slint

* callbacks fiied

* feat: setting page

* feat: change theme when app start

* chore: clean up code

* feat(ui): add scroll view in setting page

---------

Co-authored-by: Serein <[email protected]>
zwow-cj pushed a commit that referenced this pull request Aug 30, 2024
* feat:some basic laying

* feat(ui): basic laying on the page

* ui laying primary draft

* feat(ui): callbacks and root

* attempt to fill in .cc and .slint

* callbacks fiied

* feat: setting page

* feat: change theme when app start

* chore: clean up code

* feat(ui): add scroll view in setting page

---------

Co-authored-by: Serein <[email protected]>
Serein207 added a commit that referenced this pull request Sep 19, 2024
* feat(ui): complete main layout, add carousel.

* feat(ui): add layout to "current activity"

* feat(executor): different modes of `asyncExecute` using a timer

One more parameter of `asyncExecute` when using a timer: TimerFlag flag
TimerFlag::Once: perform the operation only once
TimerFlag::Periodic: perform the operation periodically
Implementation of `asyncExecuteByTimer` optimized
`io_context` exposed for more flexibility

* feat(executor): add execution strategy

* feat: add view data to `ViewManager` (#17)

* fix: added assert to `UiBridge::call`

* refactor: added included header in `BasicView` for convenience

* feat: added view data to `ViewManager`

* feat(net): github api  (#28)

* feat:completed github api  interface

* style:Change code style to match clang-format

---------

Co-authored-by: Serein <[email protected]>

* docs(executor): optimize flag description

* feat(net): evento api (#26)

* feat:Preliminarily completed the implementation of the evento api interface

* fix(net): Use `std::string` for all non-string arguments to meet the interface requirements of the URL library to resolve the compilation error.

* fix(net)!: fix the false return type of refreshAccessToken and change the return type from void to bool for some function

* fix(net): Fixed incorrect behavior in refreshAccessToken function.

* refactor(net): Modified the return value settings of some functions to make the overall behavior more in line with server interface expectations

* feat(net)!: Modified some function interfaces and implementations to align with the new backend interface specifications

* feat(net): Implemented department-related functions.

* fix(net): fix JSON parsing problem

* feat(net)!: Changed the return value types of some functions to avoid passing arguments by reference.

Additionally, renamed some incorrectly named variables in EventEntity to conform to backend interface specifications.

* style(net): Modified code style to meet requirements.

* feat(net): Add getEventList function

* feat(net)!: Deprecated some functions and change the name of some variables and functions.

* feat(net)!: Primarily enhanced the security of certain functions.

Additionally changed the definition locations of some variables and optimized the parameter names of some functions.

* feat(net): Initialize `std::optional<FeedbackEntity>` as `std::nullopt` in getUserFeedback

* feat(ui): add scrollview to layout

* feat/ui/discovery-page: add scarousel to scroollview

* feat/ui/DiscoveryPage: accomplish dynamic grid
using double for loop in VerticalLayout, but font not set yet

* feat(ui): ui core multiple update (#31)

* feat(ui): added state layer
* feat(ui): modify overlay, improve menu and login
* feat(ui): improve `MessageManager`
* feat(ui): added developer shortcut to skip `LoginOverlay`

Press Ctrl + Shift + Right click on shadow of `LoginOverlay` to skip login

* feat: set default slint style to material

* fix(ui): fix toast and login shadow (#33)

* fix(ui): make login shadow opaque
* fix(ui): make toast higher than overlay

* feat: slint experimental feature

* feat(ui): initialize sur-ui default theme

* fix: crash showing message when another message on screen

* feat(ui): optimize menu overlay

* fix: open url failed on macos

* build(deps): bump 3rdpart/sast-link-cxx-sdk from `9e2e743` to `2f62634` (#34)

Bumps [3rdpart/sast-link-cxx-sdk](https://github.com/Serein207/sast-link-cxx-sdk) from `9e2e743` to `2f62634`.
- [Release notes](https://github.com/Serein207/sast-link-cxx-sdk/releases)
- [Commits](Serein207/sast-link-cxx-sdk@9e2e743...2f62634)

---
updated-dependencies:
- dependency-name: 3rdpart/sast-link-cxx-sdk
  dependency-type: direct:production
...

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* fix: make `openBrowser` inline, added dev shortcut

Ctrl+Shift+Left click for skip login
Ctrl+Shift+Right click for fake login (not trigger onLogin)

* fix: menu expand animation lost

* chore(cmake): remove useless variable

* build(deps): bump 3rdpart/sast-link-cxx-sdk from `9e2e743` to `2f62634` (#35)

Bumps [3rdpart/sast-link-cxx-sdk](https://github.com/Serein207/sast-link-cxx-sdk) from `9e2e743` to `2f62634`.
- [Release notes](https://github.com/Serein207/sast-link-cxx-sdk/releases)
- [Commits](Serein207/sast-link-cxx-sdk@9e2e743...2f62634)

---
updated-dependencies:
- dependency-name: 3rdpart/sast-link-cxx-sdk
  dependency-type: direct:production
...

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: Serein <[email protected]>

* feat(ui): finish setting page (#21)

* feat:some basic laying

* feat(ui): basic laying on the page

* ui laying primary draft

* feat(ui): callbacks and root

* attempt to fill in .cc and .slint

* callbacks fiied

* feat: setting page

* feat: change theme when app start

* chore: clean up code

* feat(ui): add scroll view in setting page

---------

Co-authored-by: Serein <[email protected]>

* feat(ui): `LoadingButton` component (#37)

* feat: basic `LoadingButton`
* feat: add custom animate
* feat: export component in index, clean up code

* refactor!: drop icon color auto switch function (#42)

* refactor!: drop icon color auto switch
* fix: correct existing `Image`

reasons:
- reduce resource file number and executable size when embed resource
- color not exactly match environment (not the same color of text)
- auto switch require duplicate files with little difference, and not fit into some special requirements perfectly
- colorize support animation

How to migrate:

```
Image {
    source: Token.image.icon.me;
    colorize: Token.color.on-surface; // should be added to all Image with source Token.image.icon.*
}
```

Following icon keep previous behavior.

- evento (bad support for `colorize`)

BREAKING CHANGE: remove support for icon color auto-switch, please use `colorize` property of `Image` instead.

* feat(ui): event card (#38)

feat(ui): `Card`, `EventCard` , `EventCardGroup` and coversion function in `Convert.hh`

* feat(ui): improve view data logic

use std::array instead for convenience

* feat(assets): remove useless images

* feat(card): optimize animation

* feat(card): optimize event card

* feat(font): increase font size

* feat(ui)!: add md3 button (#46)

* feat(ui): added scrim, shadow color and elevation
* feat(ui): improve menu shadow
* feat(ui): improve state layer
* feat(ui): improve `LoadingAnimation`
* feat(ui): added md3 button

BREAKING CHANGE: 目前 LoadingButton 已被删除,请使用 LoadingAnimation 和 Button 的组合

* feat(ui): complete main layout, add carousel.

* feat(ui): add scrollview to layout

* feat/ui/DiscoveryPage: accomplish dynamic grid
using double for loop in VerticalLayout, but font not set yet

* feat/ui/discovery-page: use component eventcardgroup

* feat/ui/discovery-page: pull before push to remote

* feat(ui): fix compile error and conflicts

* feat/ui/discovery-page: modify component height

* feat/ui/discovery-page: finish page display

* feat/ui/discovery-page: seperate current and newest event

* feat/ui/discovery-page: delete dead code

* fix: correct about page link color

* refactor: new login overlay

* docs(readme): change to english

* feat(cache): clear disk cache

* fix(net)!: potential lifetime problem & modify api

* perf(net): change buffer to local variable

* fix(ui): load image logic

* feat(ui): clean up code

* fix

* ci: limit thread number of build

* build: replace platform macro

* fix: remove redundant code due to merge

* feat: controller logic

* fix: miss header

* perf: condition render info text

---------

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: Mairon <[email protected]>
Co-authored-by: Serein <[email protected]>
Co-authored-by: cEvolve05 <[email protected]>
Co-authored-by: shuzhuxvchuang <[email protected]>
Co-authored-by: Serein <[email protected]>
Co-authored-by: ZhiweiCOfficial <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: DTPA101 <[email protected]>
Co-authored-by: Mairon <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

feat(ui): implement setting page
3 participants