-
Notifications
You must be signed in to change notification settings - Fork 34
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
Comments
@devkanro Are contributions allowed for this issue? If yes, then I would like to work on it. |
@RivanParmar Of course, I'm now blocked by some other works, feel free to submit PRs, but don't forget to check the manual |
@devkanro Are there any guidelines on how to structure the code? Since And it would be great if discussions could be enabled for the project so that implementation ideas could be discussed over there. |
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... |
@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? |
@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 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 |
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. |
|
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 |
|
This is essentially done as of 0.5.0 |
Look and feel
Core component support
Consistency check
Consistency Issue
Project struct
The text was updated successfully, but these errors were encountered: