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: space sort & categorization #1323

Open
wants to merge 9 commits into
base: main
Choose a base branch
from

Conversation

North-West-Wind
Copy link

@North-West-Wind North-West-Wind commented Sep 9, 2024

Thank you so much for your contribution to FluffyChat ❤️❤️❤️

Please make sure that your Pull Request meet the following acceptance criteria:

  • Code formatting and import sorting has been done with dart format lib/ test/ and dart run import_sorter:main --no-comments
  • The commit message uses the format of Conventional Commits
  • The commit message describes what has been changed, why it has been changed and how it has been changed
  • Every new feature or change of the design/GUI is linked to an approved design proposal in an issue No one answered my issue (Feat: categorize subspace #1155 / feat: Room Order & Subspace Categorization #1326) so I just made this on my own
  • Every new feature in the app or the build system has a strategy how this will be tested and maintained from now on for every release, e.g. a volunteer who takes over maintainership I have never programmed in Dart/Flutter prior to this so I have absolutely no idea

Pull Request has been tested on:

  • Android
  • iOS
  • Browser (Chromium based)
  • Browser (Firefox based)
  • Browser (WebKit based)
  • Desktop Linux
  • Desktop Windows
  • Desktop macOS

Screenshot

screenshot of fluffychat on web

Additional notes

I REALLY wanted an Android Matrix client with specifically the categorization feature, but I couldn't figure out how to build an APK, so if someone can help it would be awesome.

@krille-chan
Copy link
Owner

Hey there, for design changes please first create a feature issue with low level mock-ups, then high level mock-ups and let them be approved. Otherwise you are at risk that the PR does not get accepted

@North-West-Wind
Copy link
Author

North-West-Wind commented Sep 10, 2024

What are low-level and high-level mock-ups? Are there any example issues I can refer to?

Sorry I'm just really unfamiliar with this.

@North-West-Wind
Copy link
Author

I tried to make one here: #1326

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants