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

Command creation for grass from input #13

Merged
merged 2 commits into from
Jul 11, 2016
Merged

Conversation

mayank33
Copy link
Contributor

@mayank33 mayank33 commented Jul 9, 2016

No description provided.

@mayank33
Copy link
Contributor Author

mayank33 commented Jul 9, 2016

screenshot from 2016-07-09 22 54 20

#include <Wt/WSelectionBox>


#include "Text_Gui.h"
Copy link
Owner

Choose a reason for hiding this comment

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

include only required headers

Copy link
Contributor Author

Choose a reason for hiding this comment

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

yeah, will do the code cleaning today.

@rashadkm
Copy link
Owner

rashadkm commented Jul 9, 2016

could you explain the class Text_Gui and Overall?

Also in the screenshot, there is the layout problem i mentioned on gitter.
the label and the text must come like in a horizontal layout.

easy fix will be to set all tabs use horizontal layout. label, input widget (text, checkbox) will be on each row

|Required|

Name of input vector map

Layer number or name ('-1' for all layers)

PS: bold text is a rendering by github. Not me shouting... ;)
Note that for now, there won't be any WSelectionBox inisde wgrass command module.
only textedit and checkbox

if the grass wxgui shows layername as a selectionbox, we change into simple text edit

in the bottom there on parameter called input feature type which three or four checkbox
so all the checkbox will go into a group or wcontainer widget and that goes into a single line below the label text 'Input feature type'

@mayank33
Copy link
Contributor Author

Text_Gui is a class which is made for the different options. I made that as every option hold a lot of data like name to shown in GUI, name for the grass command, type to be sent in grass command, whether it has multiple or not. These things will be used once data is entered and a grass command is to be made. Overall is combination of Text_gui and widget associated with it. Initially I made all this for easyness and also I have to access all in Mainui class so just making an another class and acessing it in MainUi is easy.

@rashadkm rashadkm merged commit 83f4f4d into rashadkm:master Jul 11, 2016
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.

None yet

2 participants