Skip to content
This repository has been archived by the owner on Apr 25, 2023. It is now read-only.

feat: widget align system #61

Merged
merged 58 commits into from
Sep 26, 2021
Merged

feat: widget align system #61

merged 58 commits into from
Sep 26, 2021

Conversation

KaWaite
Copy link
Member

@KaWaite KaWaite commented Aug 5, 2021

Overview

Front-end implementation of the widget align system

What I've done

  • Added the react-align package
  • Implemented the align system into the visualizer
  • Updated widget convert logic to split widgets based on whether they are floating(not a part of the align system(for example, the Splashscreen) widgets or not.
  • Made Button widget to replace Menu widget (legacy, and floating aka outside align system) in time
  • Updated so widgets know their position and can open menus appropriately (per each widget's implementation)

What I haven't done

Screenshot

Screen Shot 2021-08-12 at 11 39 54

Screen Shot 2021-08-12 at 11 40 05

Screen Shot 2021-08-12 at 11 40 17

Screen Shot 2021-08-12 at 11 40 30

Memo

To test:

  • Adding widgets
  • Remove widget
  • Opening align editor (click on Widgets from the left panel)
  • Move widgets from area to area
  • Re-order widgets in a single area
  • Extend a widget
  • Check Button and Storytelling's menus open correctly depending on their location
  • Change the alignment of an area's widgets(8 total areas: all center areas)
  • With editor open, clicking on another item in the left panel should close the editor
  • Make sure preview and published projects show the correct placement

@netlify
Copy link

netlify bot commented Aug 5, 2021

✔️ Deploy Preview for reearth-web ready!

🔨 Explore the source changes: 456529f

🔍 Inspect the deploy log: https://app.netlify.com/sites/reearth-web/deploys/61508ab3e122bf0008d3768b

😎 Browse the preview: https://deploy-preview-61--reearth-web.netlify.app

@codecov
Copy link

codecov bot commented Aug 5, 2021

Codecov Report

Merging #61 (456529f) into main (92cdbb4) will increase coverage by 0.32%.
The diff coverage is 87.50%.

Impacted file tree graph

@@            Coverage Diff             @@
##             main      #61      +/-   ##
==========================================
+ Coverage   57.67%   57.99%   +0.32%     
==========================================
  Files          46       46              
  Lines         730      738       +8     
  Branches      114      114              
==========================================
+ Hits          421      428       +7     
- Misses        261      262       +1     
  Partials       48       48              
Impacted Files Coverage Δ
src/theme/colors.ts 100.00% <ø> (ø)
src/theme/darkTheme.ts 100.00% <ø> (ø)
src/theme/lightheme.ts 100.00% <ø> (ø)
src/theme/theme.ts 100.00% <ø> (ø)
src/gql/graphql-client-api.tsx 53.24% <83.33%> (+0.85%) ⬆️
src/state/index.ts 77.41% <100.00%> (+1.55%) ⬆️

KaWaite added 16 commits August 6, 2021 10:10
Update widget and scene muts/query.
Add widget align sys fragment.
Add widgets as possible Mode.
Add editor toggle button w context
Update WidgetToggleButton
New widget conversion
wip: WidgetAlignSystem component
Wrap Eartheditor in editor context
Move widget functionality
Align widget functionality
Remove space between components.
Wip: Convert storytelling and menu button widgets to work with align sys
Update published Widget type.
wip: reorder functionality.
clean up align sys component.
Remove unneeded LayerIconWrapper.
Add typings to section mapping.
Make it so editor mode closes if Widgets is not selected.
wip: add extend functionality
@KaWaite KaWaite marked this pull request as ready for review August 12, 2021 02:30
@KaWaite KaWaite requested a review from rot1024 August 12, 2021 02:42
@KaWaite KaWaite requested a review from HideBa September 1, 2021 08:17
HideBa
HideBa previously approved these changes Sep 2, 2021
Copy link
Member

@HideBa HideBa left a comment

Choose a reason for hiding this comment

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

LGTM

@KaWaite KaWaite requested a review from HideBa September 8, 2021 07:51
src/plugin/api.ts Outdated Show resolved Hide resolved
src/components/organisms/EarthEditor/PropertyPane/hooks.ts Outdated Show resolved Hide resolved
src/components/organisms/EarthEditor/CanvasArea/convert.ts Outdated Show resolved Hide resolved
src/components/organisms/Published/hooks.ts Outdated Show resolved Hide resolved
@rot1024 rot1024 merged commit ed29403 into main Sep 26, 2021
@rot1024 rot1024 deleted the feat-alignsys branch September 26, 2021 15:05
keiya01 pushed a commit that referenced this pull request Apr 25, 2023
* feat: tag system domain models (#39)

* feat: tag system domain models

* refactor: * add tag interface * tag -> group and tag->item conversation

* testing: generate test cases for the tagID

* resolve notes

* fix unit tests errors

* add NewId test code
fix NewId func

* add more test cases
refactor some parts

* feat: tag system data-layer (mongo) (#44)

* feat: tag system data-layer (mongo)

* remove len > 0 check

* goimport

* Update pkg/tag/group_builder.go

Co-authored-by: rot1024 <aayhrot@gmail.com>

* Update pkg/tag/item_builder.go

Co-authored-by: rot1024 <aayhrot@gmail.com>

* rename itemFrom and groupFrom funcs

Co-authored-by: rot1024 <aayhrot@gmail.com>

* feat: create tag group and tag item (#45)

* tag item and group schema

* feat: creat tags (GQL schema)

* tag items and tag groups resolvers

* datalayer (dummy memory) and usecases

* receive list by reference

* check if nil for list

* resolve notes

* generate new models

* feat: memory infrastructure (#46)

* refactor: implement memory infrastructure

* test: implement memory infrastructure test cases

* test: fix FindByScene test case

* feat: attach/detach tag from layer (#50)

* tag item and group schema

* feat: creat tags (GQL schema)

* tag items and tag groups resolvers

* datalayer (dummy memory) and usecases

* receive list by reference

* check if nil for list

* feat: introduce tags to layers

* feat: attach/detach tags from layers

* fix imports

* refactor: resolve notes

* test: test units for tags

* refactor: resolve notes

* feat: attach/detach tag item from group (#52)

* refactor: transform group tags list to reference

* feat: attach/detach tags

* refactor: use params as use-case input

* test: mongodoc testing (#61)

* test: mongodoc testing

* resolve notes

* feat: remove tag (#58)

* feat: remove a tag (init)

* feat: remove tag

* feat: remove tag usecase and infra

* resolve notes

* feat: tag system queries (#54)

* feat: tag system queries

* resolve notes

* feat: update tag (#49)

* tag item and group schema

* feat: creat tags (GQL schema)

* tag items and tag groups resolvers

* datalayer (dummy memory) and usecases

* receive list by reference

* check if nil for list

* feat: rename tag group

* refactor: rename -> update

* resolve notes

* resolve notes

* change find by id func return type

* Merge branch 'tag-system' of https://github.com/reearth/reearth-backend into tag/update-group-label

# Conflicts:
#	internal/adapter/gql/generated.go
#	internal/adapter/gql/gqlmodel/convert_tag.go

* fix testing

* resolve notes

* resolve notes

* resolve notes

Co-authored-by: HideBa <baba.papa1120.ba@gmail.com>
Co-authored-by: rot1024 <aayhrot@gmail.com>
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants