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): implement discovery page #24

Merged
merged 63 commits into from
Sep 19, 2024
Merged

feat(ui): implement discovery page #24

merged 63 commits into from
Sep 19, 2024

Conversation

aurora0x27
Copy link
Collaborator

暂时完成页面的分区, 并添加了轮播图组件
image

@aurora0x27
Copy link
Collaborator Author

aurora0x27 commented Aug 14, 2024

f4d7bf6
这次提交在"正在进行的活动"板块增加了布局, 这个布局可以根据窗口的实际大小来决定展示几个block, 也就是之后用来放置每个活动的地方
image

@aurora0x27 aurora0x27 requested review from cEvolve05 and removed request for ArcticLampyrid August 14, 2024 08:28
CMakePresets.json Outdated Show resolved Hide resolved
ui/views/page/discovery.slint Show resolved Hide resolved
@Serein207 Serein207 added the feature New feature or request label Aug 14, 2024
@Serein207 Serein207 linked an issue Aug 14, 2024 that may be closed by this pull request
@Serein207
Copy link
Member

Serein207 commented Aug 14, 2024

  • 未完成的 PR 改成 draft
  • 有空把 dev 分支的提交合并过来

@ArcticLampyrid ArcticLampyrid marked this pull request as draft August 14, 2024 10:26
Copy link
Member

@cEvolve05 cEvolve05 left a comment

Choose a reason for hiding this comment

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

页面整体应当进行垂直布局,同时不要用横向滑动的方式隐藏显示不下的内容,存在空间不足的时候应当换行解决

ui/views/page/discovery.slint Outdated Show resolved Hide resolved
ui/views/page/discovery.slint Outdated Show resolved Hide resolved
aurora0x27 and others added 8 commits August 15, 2024 17:48
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
* fix: added assert to `UiBridge::call`

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

* feat: added view data to `ViewManager`
* feat:completed github api  interface

* style:Change code style to match clang-format

---------

Co-authored-by: Serein <[email protected]>
* 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
Copy link

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

@Serein207
Copy link
Member

活动卡片上的活动图片已不再需要展示

Copy link

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

cEvolve05 and others added 3 commits August 29, 2024 10:27
* 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
* fix(ui): make login shadow opaque
* fix(ui): make toast higher than overlay
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/Infrastructure/Network/NetworkClient.h Outdated Show resolved Hide resolved
src/Infrastructure/Network/NetworkClient.h Outdated Show resolved Hide resolved
Copy link

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

Copy link

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

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 September 19, 2024 14:36
@Serein207 Serein207 merged commit e2c2e24 into dev Sep 19, 2024
5 checks passed
@Serein207 Serein207 deleted the feat/ui/discovery-page branch September 19, 2024 14:55
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 discovery page
8 participants