Skip to content

Fixes FreeRDP/Remmina#473 - Customize button in the connection editor should show the resolutions dialog#501

Merged
antenore merged 4 commits into
FreeRDP:nextfrom
antenore:next
Mar 2, 2015
Merged

Fixes FreeRDP/Remmina#473 - Customize button in the connection editor should show the resolutions dialog#501
antenore merged 4 commits into
FreeRDP:nextfrom
antenore:next

Conversation

@antenore
Copy link
Copy Markdown
Member

@muflone it works, but I'm not sure about the quality of the code. Can you please review it?

Thanks in advance

Comment thread remmina/src/remmina_file_editor.c Outdated
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

This thing, duplicated between preferences dialog and file editor could be moved in remmina_public to avoid code duplication.

@muflone
Copy link
Copy Markdown
Contributor

muflone commented Feb 25, 2015

This is great, @antenore
Could you please note the previous comments above?

@antenore
Copy link
Copy Markdown
Member Author

Thanks @muflone . I'll work on it the next days.

@muflone
Copy link
Copy Markdown
Contributor

muflone commented Feb 28, 2015

@antenore
I'm going to make some changes to the string_list code (two columns string list for description/values), including the remmina_string_list_new.
Could you please make the previous changes and commit this PR or you will need to solve a conflict later

@antenore
Copy link
Copy Markdown
Member Author

Don't worry @muflone , go ahead, . I'll solve the conflicts.

I still cannot work on Remmina for some days.

antenore added 2 commits March 2, 2015 09:54
 Added translations to ignore certificate
 Added cert_ignore for RDP plugin
 Don't set gateway_usage_method for non RD gateway connections
 Added second column data string_list, use two_columns argument during the creation
 Removed samples data
 Added RD Gateway server setting
 Code cleanup
 Restored icon in the external tools menu item
 Merge pull request #505 from FreeRDP/Issues/503
 Code cleanup
 Added a new preference to show or hide the menu icons
 Added a new preference to show or hide the buttons icons
 Renamed preferences widgets from options to appearance
 Added appearance page in preferences dialog
 Merge pull request #500 from KyleSanderson/memleak
 Merge pull request #502 from ozbek/next
 Add translations for Uzbek Cyrillic
 Fix small memory leaks and a NULL ptr dereference.
@antenore
Copy link
Copy Markdown
Member Author

antenore commented Mar 2, 2015

@muflone , @giox069

Back to business, can you kindly take a look please.

@muflone
Copy link
Copy Markdown
Contributor

muflone commented Mar 2, 2015

Seems fine to me, you can merge it

antenore added a commit that referenced this pull request Mar 2, 2015
Fixes FreeRDP/Remmina#473 - Customize button in the connection editor should  show the resolutions dialog
@antenore antenore merged commit 681ca9e into FreeRDP:next Mar 2, 2015
antenore added a commit that referenced this pull request Jul 10, 2015
Fixes FreeRDP/Remmina#473 - Customize button in the connection editor should  show the resolutions dialog
antenore added a commit that referenced this pull request Jul 10, 2015
Fixes FreeRDP/Remmina#473 - Customize button in the connection editor should  show the resolutions dialog
antenore added a commit that referenced this pull request Jul 10, 2015
Fixes FreeRDP/Remmina#473 - Customize button in the connection editor should  show the resolutions dialog
antenore added a commit that referenced this pull request Jul 10, 2015
Fixes FreeRDP/Remmina#473 - Customize button in the connection editor should  show the resolutions dialog
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants