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
Add scaling parameter #9493
base: master
Are you sure you want to change the base?
Add scaling parameter #9493
Conversation
Can one of the admins verify this patch? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Overall ok, but some things missing.
also, please run git clang-format
before committing a change, it will fail the ci builder otherwise.
@@ -38,6 +38,7 @@ public class BookmarkDB extends SQLiteOpenHelper | |||
static final String DB_KEY_SCREEN_RESOLUTION = "resolution"; | |||
static final String DB_KEY_SCREEN_WIDTH = "width"; | |||
static final String DB_KEY_SCREEN_HEIGHT = "height"; | |||
static final String DB_KEY_SCREEN_SCALE = "scale"; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
you need to increment the database version and add an update path for the new field.
android:entries="@array/scale_array" | ||
android:entryValues="@array/scale_values_array" | ||
android:key="bookmark.scale" | ||
android:summary="Scaling Percentage" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
- no hard coded strings, move to
strings.xml
. - Summary is not really helpful, this is
HighDPI
support, so remote scaling, not local/smart-sizing
like
android:entryValues="@array/scale_values_array" | ||
android:key="bookmark.scale" | ||
android:summary="Scaling Percentage" | ||
android:title="@string/settings_scale" /> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
missing in strings.xml
This is how are pull requests handled by FreeRDP
Preparations before creating a pull
.clang-format
clangformat
reformats the whole codebaseTo ease accepting your contribution
What you should be prepared for
Please remove this text before submitting your pull!