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

Adding a open source L&F to ArgoUML #9

Open
vincenzopalazzo opened this issue May 15, 2020 · 9 comments
Open

Adding a open source L&F to ArgoUML #9

vincenzopalazzo opened this issue May 15, 2020 · 9 comments

Comments

@vincenzopalazzo
Copy link
Member

Hi all,

I want to ask if PR that adds new L&F is welcome.

I'm playing with ArgoUML code and I added a material-ui-swing look and feel to test the project with a big and complex client app.

Selection_078

@stcarrez
Copy link
Contributor

Yes, that could be a nice idea!

@vincenzopalazzo
Copy link
Member Author

Hi @stcarrez,

Thanks, I will work on the library official release and I will start the work on the ArgoUML l&f integration.

@linustolke
Copy link
Member

Contributions are welcome. We don't use Pull Requests but instead use repo/Gerrithub.

@vincenzopalazzo
Copy link
Member Author

Hi @linustolke,

Thank you. Yes, I wrote PR I'm sorry, but I wanted mean to submit my change in repo/Gerrithub.

@vincenzopalazzo
Copy link
Member Author

vincenzopalazzo commented Jul 11, 2020

Hi @linustolke and @stcarrez,

I was starting to work to test material-ui-swing library on ArgoUML and I have a demo version, with incomplete code, but I want share the code with the team because it knows the code better than me and I think that I can receiver good feedbacks about improving the code.

I don't know gerrithub and I don't know what happens if I push my actual version of code on my branch, it remains on my branch or the push start the auto review process??

Thanks for your time.

P.S: another screenshot of the style is in the photo below, and also exist this issue yet

@stcarrez
Copy link
Contributor

I'm interested to try your version (either from your own git repo or from a repo branch).

Is it possible to increase the font size in your implementation?
Having bigger text and bigger icons is something I'm really interested in because I have problems when
using the current ArgoUML.

Have a look at https://github.com/argouml-tigris-org/argouml/wiki/Working-in-the-project
I was able to use repo as explained, build, test and so on, but the submission with repo upload failed for me, even though I have an account on gerrit.

For the review process, @linustolke will tell you.

@vincenzopalazzo
Copy link
Member Author

Hi @stcarrez,

I'm interested to try your version (either from your own git repo or from a repo branch).
I used repo to clone the project, but I don't know if I push my branch, it will available oonline. I don't have any github repository with the code but I can generate a build for you.

There is a few bugs on GUI, the most evident is in the toolbar with a tool to build the diagram, but at this link, there is my local version.

Having bigger text and bigger icons is something I'm really interested in because I have problems when
using the current ArgoUML.

I think that the problem with the font is resolved also with metal theme? (maybe) but I think that with L&F is possible to change the font size but I don't know if is possible to make the change with image size, because I don't know the format (png/jpg) of these images.

@linustolke
Copy link
Member

Hello @vincenzopalazzo!

The main idea with using repo/GerritHub is to get a clean line of changes on master. Every change is a single commit (not a merge commit) on master with the complete change. The upload should be a single commit that is then amended with preserved ChangeID.

If keeping your branch means that you will:

  • Upload multiple commits to GerritHub, that fixes problems back and forth, that will be cumbersome because each commit is then considered as a separate change and to keep the master change log clean I would not want to approve a commit that is soon fixed.
  • Not keep the same ChangeID between the times you upload to GerritHub, there will be a lot of confusion. Each upload will then be considered as a new change and it will not be possible to use GerritHub to trace comments, if they are fixed, and compare different uploads.

@vincenzopalazzo
Copy link
Member Author

Hi @linustolke,

Sorry for the delay.

Thanks for the clarification about how GerritHub, and if I understand is not good to talk about the code but is good to propose a new feature or bug fix

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

No branches or pull requests

3 participants