UI: fix and improve a few messages #110321

Merged
Hans Goudey merged 2 commits from pioverfour/blender:dp_fix_messages into main 2023-07-24 21:22:14 +02:00
Member
  • "Rename Channels": only one channel can be renamed at a time, use
    singular.
  • "Copy Markers to Scene": rephrase erroneous operator description.
  • "Axis tag names with": grammar.
  • "Close or open the selected stroke adding an edge from last to first
    point": "segment" is preferred to "edge" elsewhere in the context of
    curves or Grease Pencil.
  • "Number of subdivisions [points] by edge[s]": replace "by edge" with
    "per segment" for the same reason.
  • "Compatibility mode for SL, OpenSim...": expand to explain that SL
    and OpenSim are respectively Second Life and OpenSimulator.
  • "W/m^2" -> "W/m²". This symbol is widely supported and should be
    preferred for exponentiation outside of code.
  • "Effect on tracks which are tracked less than specified amount of frames",
    "Effect on tracks which have a larger reprojection error":
    Use "Affect" and "number" instead of "amount".
  • "Hull curve" -> "Envelope". This is a calque from German Hüllkurve
    meaning envelope.
  • "Frquency Cutoff" -> "Frequency Cutoff" (typo)
  • "Check if Select Left or Right": rephrase to "Based on Mouse
    Position" as it better explains the action of the operator.
  • "Make cut event if strip is not selected ..." -> even (typo)
  • "Shear selected items along the horizontal screen axis":
    Rephrase as this transform operator can act in many different axes.
  • Tonemapping compositing node: the two algorithms "R/D Photoreceptor"
    and "Rh Simple" only had names, but no description. Add ones
    explaining at least the basic principle and where the names come
    from.
  • In the "Matte dilate/erode side" description for the Keying node's
    Dilate/Erode socket, "side" was likely a typo for "size".
    Reformulate the description to make it clearer, inspired by similar
    ones.
  • "Width of the blur edge" -> "Width of the blur for the transition";
    "Edge angle" -> "Angle of the transition";
    "Wipe direction" -> "Whether to fade in or out":
    Better explains the sequencer wipe transition (inspired by the manual).
  • OSL shaders now supported on some GPU backends.
  • "Add a new repeat input and output nodes " -> "Add new" (typo).
- "Rename Channels": only one channel can be renamed at a time, use singular. - "Copy Markers to Scene": rephrase erroneous operator description. - "Axis tag names with": grammar. - "Close or open the selected stroke adding an edge from last to first point": "segment" is preferred to "edge" elsewhere in the context of curves or Grease Pencil. - "Number of subdivisions [points] by edge[s]": replace "by edge" with "per segment" for the same reason. - "Compatibility mode for SL, OpenSim...": expand to explain that SL and OpenSim are respectively Second Life and OpenSimulator. - "W/m^2" -> "W/m²". This symbol is widely supported and should be preferred for exponentiation outside of code. - "Effect on tracks which are tracked less than specified amount of frames", "Effect on tracks which have a larger reprojection error": Use "Affect" and "number" instead of "amount". - "Hull curve" -> "Envelope". This is a calque from German Hüllkurve meaning envelope. - "Frquency Cutoff" -> "Frequency Cutoff" (typo) - "Check if Select Left or Right": rephrase to "Based on Mouse Position" as it better explains the action of the operator. - "Make cut event if strip is not selected ..." -> even (typo) - "Shear selected items along the horizontal screen axis": Rephrase as this transform operator can act in many different axes. - Tonemapping compositing node: the two algorithms "R/D Photoreceptor" and "Rh Simple" only had names, but no description. Add ones explaining at least the basic principle and where the names come from. - In the "Matte dilate/erode side" description for the Keying node's Dilate/Erode socket, "side" was likely a typo for "size". Reformulate the description to make it clearer, inspired by similar ones. - "Width of the blur edge" -> "Width of the blur for the transition"; "Edge angle" -> "Angle of the transition"; "Wipe direction" -> "Whether to fade in or out": Better explains the sequencer wipe transition (inspired by the manual). - OSL shaders now supported on some GPU backends. - "Add a new repeat input and output nodes " -> "Add new" (typo).
Damien Picard added the
Module
User Interface
label 2023-07-20 22:28:21 +02:00
Damien Picard force-pushed dp_fix_messages from fd94015b08 to c488cb56dc 2023-07-20 22:36:57 +02:00 Compare
Damien Picard reviewed 2023-07-20 22:40:12 +02:00
@ -3220,3 +3220,3 @@
{
/* identifiers */
ot->name = "Rename Channels";
ot->name = "Rename Channel";
Author
Member

Only one channel can be renamed at a time, use singular.

Only one channel can be renamed at a time, use singular.
@ -1804,3 +1804,3 @@
/* identifiers */
ot->name = "Make Links to Scene";
ot->name = "Copy Markers to Scene";
Author
Member

Rephrase erroneous operator description: the markers are actually copied, no link is preserved.

Rephrase erroneous operator description: the markers are actually copied, no link is preserved.
@ -610,3 +610,3 @@
/* settings */
ot->prop = RNA_def_enum(ot->srna, "type", axis_items, 0, "Axis", "Axis tag names with");
ot->prop = RNA_def_enum(ot->srna, "type", axis_items, 0, "Axis", "Axis to tag names with");
Author
Member

Grammar.

Grammar.
@ -615,3 +615,3 @@
/* settings */
ot->prop = RNA_def_enum(ot->srna, "axis", axis_items, 0, "Axis", "Axis tag names with");
ot->prop = RNA_def_enum(ot->srna, "axis", axis_items, 0, "Axis", "Axis to tag names with");
Author
Member

Grammar.

Grammar.
@ -3372,3 +3372,3 @@
ot->name = "Set Cyclical State";
ot->idname = "GPENCIL_OT_stroke_cyclical_set";
ot->description = "Close or open the selected stroke adding an edge from last to first point";
ot->description = "Close or open the selected stroke adding a segment from last to first point";
Author
Member

"segment" is preferred to "edge" elsewhere in the context of curves or Grease Pencil.

"segment" is preferred to "edge" elsewhere in the context of curves or Grease Pencil.
@ -2026,3 +2026,3 @@
MAX_EDGES,
"Subdivisions",
"Number of subdivision by edges",
"Number of subdivisions per segment",
Author
Member

Replace "by edge" with "per segment" for the same reason ("segment" is preferred to "edge" elsewhere in the context of curves or Grease Pencil).

Replace "by edge" with "per segment" for the same reason ("segment" is preferred to "edge" elsewhere in the context of curves or Grease Pencil).
@ -669,0 +666,4 @@
"open_sim",
0,
"Export to SL/OpenSim",
"Compatibility mode for Second Life, OpenSimulator and other compatible online worlds");
Author
Member

Expand to explain that SL and OpenSim are respectively Second Life and OpenSimulator.

Expand to explain that SL and OpenSim are respectively Second Life and OpenSimulator.
@ -1764,2 +1764,2 @@
"Effect on tracks which are tracked less than "
"specified amount of frames",
"Affect tracks which are tracked less than the "
"specified number of frames",
Author
Member

Use "Affect" and "number" instead of "amount".

Use "Affect" and "number" instead of "amount".
@ -1772,3 +1772,3 @@
FLT_MAX,
"Reprojection Error",
"Effect on tracks which have got larger reprojection error",
"Affect tracks which have got larger reprojection error",
Author
Member

"Affect" is the right verb here.

"Affect" is the right verb here.
Member

Adding on, "have got" is informal and sounds wrong as UI text, better remove "got" here

Adding on, "have got" is informal and sounds wrong as UI text, better remove "got" here
@ -1250,3 +1250,3 @@
2.0,
"Attack Time",
"Value for the hull curve calculation that tells how fast the hull curve can rise "
"Value for the envelope calculation that tells how fast the envelope can rise "
Author
Member

This is a calque from German Hüllkurve meaning envelope.

This is a calque from German Hüllkurve meaning envelope.
@ -1468,3 +1468,3 @@
0.0f,
FLT_MAX,
"Frquency Cutoff (Hz)",
"Frequency Cutoff (Hz)",
Author
Member

Typo.

Typo.
@ -437,3 +437,3 @@
/* defines for left-right select tool */
static const EnumPropertyItem prop_nlaedit_leftright_select_types[] = {
{NLAEDIT_LRSEL_TEST, "CHECK", 0, "Check if Select Left or Right", ""},
{NLAEDIT_LRSEL_TEST, "CHECK", 0, "Based on Mouse Position", ""},
Author
Member

Rephrase to better explain the action of the operator.

Rephrase to better explain the action of the operator.
@ -1590,3 +1590,3 @@
false,
"Ignore Selection",
"Make cut event if strip is not selected preserving selection state after cut");
"Make cut even if strip is not selected preserving selection state after cut");
Author
Member

Typo.

Typo.
@ -1029,3 +1029,3 @@
/* identifiers */
ot->name = "Shear";
ot->description = "Shear selected items along the horizontal screen axis";
ot->description = "Shear selected items along the given axis";
Author
Member

Rephrase as this transform operator can act in many different axes.

Rephrase as this transform operator can act in many different axes.
@ -186,3 +186,3 @@
RNA_def_property_ui_range(prop, 0.0f, 10.0f, 1, 3);
RNA_def_property_ui_text(
prop, "Strength", "Sunlight strength in watts per meter squared (W/m^2)");
prop, "Strength", "Sunlight strength in watts per meter squared (W/m²)");
Author
Member

This symbol is widely supported and should be preferred for exponentiation outside of code.

This symbol is widely supported and should be preferred for exponentiation outside of code.
@ -7880,0 +7880,4 @@
0,
"R/D Photoreceptor",
"More advanced algorithm based on eye physiology, by Reinhard and Devlin"},
{0, "RH_SIMPLE", 0, "Rh Simple", "Simpler photographic algorithm by Reinhard"},
Author
Member

Tonemapping compositing node: the two algorithms "R/D Photoreceptor" and "Rh Simple" only had names, but no description. Add ones explaining at least the basic principle and where the names come from.

Tonemapping compositing node: the two algorithms "R/D Photoreceptor" and "Rh Simple" only had names, but no description. Add ones explaining at least the basic principle and where the names come from.
@ -8764,3 +8768,3 @@
RNA_def_property_int_sdna(prop, nullptr, "dilate_distance");
RNA_def_property_range(prop, -100, 100);
RNA_def_property_ui_text(prop, "Dilate/Erode", "Matte dilate/erode side");
RNA_def_property_ui_text(prop, "Dilate/Erode", "Distance to grow/shrink the matte");
Author
Member

In the "Matte dilate/erode side" description for the Keying node's Dilate/Erode socket, "side" was likely a typo for "size".
Reformulate the description to make it clearer, inspired by similar ones.

In the "Matte dilate/erode side" description for the Keying node's Dilate/Erode socket, "side" was likely a typo for "size". Reformulate the description to make it clearer, inspired by similar ones.
@ -3053,1 +3053,3 @@
prop, "Blur Width", "Width of the blur edge, in percentage relative to the image size");
prop,
"Blur Width",
"Width of the blur for the transition, in percentage relative to the image size");
Author
Member

Better explain the sequencer wipe transition (inspired by the manual).

Better explain the sequencer wipe transition (inspired by the manual).
@ -3056,3 +3058,3 @@
prop = RNA_def_property(srna, "angle", PROP_FLOAT, PROP_ANGLE);
RNA_def_property_range(prop, DEG2RADF(-90.0f), DEG2RADF(90.0f));
RNA_def_property_ui_text(prop, "Angle", "Edge angle");
RNA_def_property_ui_text(prop, "Angle", "Angle of the transition");
Author
Member

Better explain the sequencer wipe transition (inspired by the manual).

Better explain the sequencer wipe transition (inspired by the manual).
@ -3062,3 +3064,3 @@
RNA_def_property_enum_sdna(prop, nullptr, "forward");
RNA_def_property_enum_items(prop, wipe_direction_items);
RNA_def_property_ui_text(prop, "Direction", "Wipe direction");
RNA_def_property_ui_text(prop, "Direction", "Whether to fade in or out");
Author
Member

Better explain the sequencer wipe transition (inspired by the manual).

Better explain the sequencer wipe transition (inspired by the manual).
@ -92,3 +92,3 @@
DefNode(ShaderNode, SH_NODE_NORMAL_MAP, def_sh_normal_map, "NORMAL_MAP", NormalMap, "Normal Map", "Generate a perturbed normal from an RGB normal map image. Typically used for faking highly detailed surfaces")
DefNode(ShaderNode, SH_NODE_TANGENT, def_sh_tangent, "TANGENT", Tangent, "Tangent", "Generate a tangent direction for the Anisotropic BSDF")
DefNode(ShaderNode, SH_NODE_SCRIPT, def_sh_script, "SCRIPT", Script, "Script", "Generate an OSL shader from a file or text data-block.\nNote: OSL shaders are not supported on the GPU")
DefNode(ShaderNode, SH_NODE_SCRIPT, def_sh_script, "SCRIPT", Script, "Script", "Generate an OSL shader from a file or text data-block.\nNote: OSL shaders are not supported on all GPU backends")
Author
Member

OSL shaders are now supported on some GPU backends.

OSL shaders are now supported on some GPU backends.
@ -121,3 +121,3 @@
AddNodeItem item;
item.ui_name = IFACE_("Repeat Zone");
item.description = TIP_("Add a new repeat input and output nodes to the node tree");
item.description = TIP_("Add new repeat input and output nodes to the node tree");
Author
Member

Typo.

Typo.
Damien Picard force-pushed dp_fix_messages from c488cb56dc to 94ffb2cdcf 2023-07-24 16:10:09 +02:00 Compare
Damien Picard requested review from Hans Goudey 2023-07-24 16:14:34 +02:00
Hans Goudey approved these changes 2023-07-24 20:15:10 +02:00
Hans Goudey left a comment
Member

Just left one reply with another suggestion. Other than that looks good, thanks!

Just left one reply with another suggestion. Other than that looks good, thanks!
Damien Picard added 1 commit 2023-07-24 21:06:53 +02:00
Author
Member

Adding on, "have got" is informal and sounds wrong as UI text, better remove "got" here

Actually noticed that it should read “tracks which have a larger”, with the article.

> Adding on, "have got" is informal and sounds wrong as UI text, better remove "got" here Actually noticed that it should read “tracks which have **a** larger”, with the article.
Hans Goudey merged commit 13ec961c4a into main 2023-07-24 21:22:14 +02:00
Damien Picard deleted branch dp_fix_messages 2023-07-24 21:29:10 +02:00
Sign in to join this conversation.
No reviewers
No Label
Interest
Alembic
Interest
Animation & Rigging
Interest
Asset Browser
Interest
Asset Browser Project Overview
Interest
Audio
Interest
Automated Testing
Interest
Blender Asset Bundle
Interest
BlendFile
Interest
Collada
Interest
Compatibility
Interest
Compositing
Interest
Core
Interest
Cycles
Interest
Dependency Graph
Interest
Development Management
Interest
EEVEE
Interest
EEVEE & Viewport
Interest
Freestyle
Interest
Geometry Nodes
Interest
Grease Pencil
Interest
ID Management
Interest
Images & Movies
Interest
Import Export
Interest
Line Art
Interest
Masking
Interest
Metal
Interest
Modeling
Interest
Modifiers
Interest
Motion Tracking
Interest
Nodes & Physics
Interest
OpenGL
Interest
Overlay
Interest
Overrides
Interest
Performance
Interest
Physics
Interest
Pipeline, Assets & IO
Interest
Platforms, Builds & Tests
Interest
Python API
Interest
Render & Cycles
Interest
Render Pipeline
Interest
Sculpt, Paint & Texture
Interest
Text Editor
Interest
Translations
Interest
Triaging
Interest
Undo
Interest
USD
Interest
User Interface
Interest
UV Editing
Interest
VFX & Video
Interest
Video Sequencer
Interest
Virtual Reality
Interest
Vulkan
Interest
Wayland
Interest
Workbench
Interest: X11
Legacy
Blender 2.8 Project
Legacy
Milestone 1: Basic, Local Asset Browser
Legacy
OpenGL Error
Meta
Good First Issue
Meta
Papercut
Meta
Retrospective
Meta
Security
Module
Animation & Rigging
Module
Core
Module
Development Management
Module
EEVEE & Viewport
Module
Grease Pencil
Module
Modeling
Module
Nodes & Physics
Module
Pipeline, Assets & IO
Module
Platforms, Builds & Tests
Module
Python API
Module
Render & Cycles
Module
Sculpt, Paint & Texture
Module
Triaging
Module
User Interface
Module
VFX & Video
Platform
FreeBSD
Platform
Linux
Platform
macOS
Platform
Windows
Priority
High
Priority
Low
Priority
Normal
Priority
Unbreak Now!
Status
Archived
Status
Confirmed
Status
Duplicate
Status
Needs Info from Developers
Status
Needs Information from User
Status
Needs Triage
Status
Resolved
Type
Bug
Type
Design
Type
Known Issue
Type
Patch
Type
Report
Type
To Do
No Milestone
No project
No Assignees
2 Participants
Notifications
Due Date
The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: blender/blender#110321
No description provided.