P & S preset rotate and hotkey #28
base: master
Are you sure you want to change the base?
Conversation
Commit ba56ce2 is dirty, it removes a build configuration from the solution file. |
I don't think MSVC works well with UTF-8 resource files, Underground78 or ��XhmikosR� can tell you more about that. |
Its correct, resource files need to either be ASCII, or if you need more then the 7-bit ASCII provides, UTF-16. |
Exactly, UTF8 doesn't work. |
And as we have already discussed in the past, people didn't like the delete option. Personally, I see when it can be useful, but it's what most people prefer. (Re-opening the PR since I accidentally closed it) So, clean up your patches, and force push. john-peterson@a47923c and john-peterson@3a30d1f should be removed completely from the PR. And please run |
@@ -225,7 +228,6 @@ | |||
#define ID_FAVORITES_ORGANIZE 937 | |||
#define ID_FAVORITES_ADD 938 | |||
#define ID_HELP_HOMEPAGE 939 | |||
//#define ID_HELP_DOCUMENTATION 940 |
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.
Don't remove this in the same patch please. It's a leftover on purpose but I'd rather completely remove it from every file than partially.
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.
I'm aware of this, but that's why you are supposed to verify the changes before you push. Don't remove these for now.
General comment about commit messages: While in general your commit messages are nice, I would try to keep commit messages' line lengths under control, especially for the summary line -- preferably somewhere between 50 to around 70 characters. After that you can add an empty line and then start the possible extra paragraphs for more detailed explanation etc. with a line length of around around 72-75 characters. This makes you create a short yet descriptive description/summary on the first line, and then lets you express yourself in a longer way after that. A similar style is mentioned here, but I prefer to have the possibility of having up to 70 or so characters in the summary. |
|
We've changed the resource file encoding to ASCII while keeping the translations in UCS-2 and updating the translation scripts to handle this. So far it's working well, you can remove e28d17229a0334d051c5f0014082e4401c2b922e. |
Adding P & S preset OSD message
Reading rotate P & S preset
Discussed at
@Underground78
Read three additional PnS parameters from [Settings\PnSPresets], if they are present
I use it for horizontal and vertical flip
No
No it's not saved by the P & S dialog because
Adding P & S preset toggle key
Name and position
The key
name is "Next PnS Preset" because
position is near "Next AR Preset" because
The key is
renamed to "PnS Next Preset" because
moved near the other "PnS" keys because
Previous PnS Preset
Resource
Discussed at #28 (comment)
This line shouldn't be kept
because
String Table → Open
orResource Symbols
dialogChanging resource file encoding from UCS-2 to ASCII
Discussed at #28 (comment), #28 (comment), #28 (comment)
When a UCS-2 file is changed the patch with the change will be binary if one file version (in this case the old) is binary. (Subsequent edits will be from text to text.)
The disadvantage of a binary patch is
git create a text patch for UTF-8 and ASCII
Programs that support additional character sets (and therefore don't rely on git to generate the patch) include
Is there another way to solve the problem?
Fixing ffmpeg gcc i686-w64 host prefix
This commit is in mingw
It's not merged because
Commit message
@jeeb
I agree that the first line should be a title of this length
The author shouldn't create line breaks in the commit message because