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

Refactoring New UI implementation based on core components #28

Closed
11 of 21 tasks
devkanro opened this issue Dec 21, 2022 · 12 comments · Fixed by #30
Closed
11 of 21 tasks

Refactoring New UI implementation based on core components #28

devkanro opened this issue Dec 21, 2022 · 12 comments · Fixed by #30
Assignees
Labels
feature New feature or request

Comments

@devkanro
Copy link
Collaborator

devkanro commented Dec 21, 2022

Look and feel

  • Palettes Basic colors have been added, it will be changed in future
  • Painters SVGs have been redesigned for Checkbox and Radio button, it does not copy from IJ because there are many temp icons in 22.3
  • Metrics Just copy from Darcula theme, need check and adjust
  • Typography Change the default font to Inter

Core component support

Consistency check

Consistency Issue

Project struct

  • Remove old impl
  • Grallery app
@devkanro devkanro self-assigned this Dec 21, 2022
@devkanro devkanro added the feature New feature or request label Dec 21, 2022
This was linked to pull requests Dec 21, 2022
@devkanro devkanro reopened this Dec 22, 2022
@RivanParmar
Copy link
Contributor

@devkanro Are contributions allowed for this issue? If yes, then I would like to work on it.

@devkanro
Copy link
Collaborator Author

devkanro commented Apr 6, 2023

@RivanParmar Of course, I'm now blocked by some other works, feel free to submit PRs, but don't forget to check the manual

@RivanParmar
Copy link
Contributor

RivanParmar commented Apr 7, 2023

@devkanro Are there any guidelines on how to structure the code? Since Checkbox, RadioButton and others are structured differently than each other. Also where should I look for implementation reference?

And it would be great if discussions could be enabled for the project so that implementation ideas could be discussed over there.

@rock3r
Copy link
Collaborator

rock3r commented Apr 7, 2023

The guiding principle is to try and do as close to Material as possible, because that API has been iterated and battle tested a lot, and also folks are familiar with it.

And yes, this means we'll have to go back and tweak existing code in this repo...

@RivanParmar
Copy link
Contributor

@rock3r What is the difference between a TextField and a TextArea? If it's same except for the number of lines, then would it be okay to implement TextField and TextArea in the same file with a common base?

@rock3r
Copy link
Collaborator

rock3r commented Apr 8, 2023

@RivanParmar they are very similar in functionality and the main difference is that an area is multiline and a field is not. I think it's ok for now to put them in the same file. The only thing I'd recommend is to check the swing implementation of the two and compare if they also look the same. You can use the UI inspector for that (details in the readme). The alternative is looking at the various Darcula*UI code and seeing how they work. There may be slight cosmetic differences between the two.

@RivanParmar RivanParmar mentioned this issue Apr 9, 2023
@devkanro
Copy link
Collaborator Author

@RivanParmar TextArea has an footer message about history, but I can't be sure if this feature is still preserved in the new ui...

https://jetbrains.design/intellij/controls/text_area/#history

@c5inco
Copy link
Collaborator

c5inco commented Apr 10, 2023

I have actually not seen that history feature used consistently in TextAreas in the IDE. I would not worry about having it implemented yet, as it can easily be composed together with the field itself later, and I think we should reconsider whether it should be coupled this way.

@devkanro
Copy link
Collaborator Author

image
@c5inco But these footer messages still can be found in the settings of version 23.1

@c5inco
Copy link
Collaborator

c5inco commented Apr 10, 2023

Sure but I'm not sure it needs to be baked in just yet. We can approach the architecture of these components differently now that we're using Compose

@devkanro
Copy link
Collaborator Author

image
@c5inco you are right, the history has been moved to outside of text area, users can simply compose lable and text area to impl it.

@rock3r rock3r closed this as completed Sep 7, 2023
@rock3r
Copy link
Collaborator

rock3r commented Sep 7, 2023

This is essentially done as of 0.5.0

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
Status: Done
4 participants