Page MenuHome

Peter Fog (tintwotin)
User

Projects

User does not belong to any projects.

User Details

User Since
Nov 24 2017, 7:42 PM (90 w, 5 d)

Recent Activity

Mon, Aug 19

Peter Fog (tintwotin) added a comment to T68512: blender 2.8 malfunctions when reporting an indentation error executing a python script.

Quite strangely everything works correctly when trying that line of code, but if I restart Blender and do the following I get the non descriptive indentation error:

Mon, Aug 19, 11:14 PM · BF Blender
Peter Fog (tintwotin) updated subscribers of T68512: blender 2.8 malfunctions when reporting an indentation error executing a python script.

Is it ok I tag you, as Text Editor module owner, on this report @Campbell Barton (campbellbarton)?

Mon, Aug 19, 3:37 PM · BF Blender
Peter Fog (tintwotin) created D5525: Clip Editor: Move fractional zoom values into sub-menu..
Mon, Aug 19, 9:23 AM · BF Blender

Sun, Aug 18

Peter Fog (tintwotin) added a comment to D5047: UI: Dim Text Editor Caret and Selections When Inactive.

Has this been committed yet? Maybe in addition, it could be considered to give the Text Editor a typing focus, like the text entry boxes?

Sun, Aug 18, 12:32 PM · User Interface
Peter Fog (tintwotin) added a comment to D5517: Text Editor: Rename Text menu to File menu.

Yeah, it's properly part if of a larger "normalization" process, which at some point should also include Menu re-organization. Having View as first menu, is unique to Blender too. And having to use a menu called Add to import files is pretty unique too(like in the VSE).

Sun, Aug 18, 11:47 AM
Peter Fog (tintwotin) added a comment to D5517: Text Editor: Rename Text menu to File menu.

Would rather leave this as is.
It means documentation/tutorials cannot say "Click on the File menu" without some risk the users gets the wrong file menu.

Sun, Aug 18, 10:15 AM
Peter Fog (tintwotin) created D5517: Text Editor: Rename Text menu to File menu.
Sun, Aug 18, 8:31 AM

Sat, Aug 17

Peter Fog (tintwotin) added a comment to rBM5719: Sequencer: Text-Strip Panel (2.81).

The alignment tooltips maybe needs to be corrected too. The icons cause confusion because, the operator is not working like in a text editor, but relative to the location(as described it this patch).

Sat, Aug 17, 12:53 PM
Peter Fog (tintwotin) created D5512: Video Editor: Move Effect panel above Adjust panel in Sidebar.
Sat, Aug 17, 11:21 AM · BF Blender
Peter Fog (tintwotin) added a comment to T68747: Audio Will not play in Video Sequence Editor.

Your .blend file does not contain any source files. However, in your .blend, if I change the source to an audio file on my computer, the audio is playing fine, so there is no setting in the file which might cause not to hear the audio.

Sat, Aug 17, 8:56 AM · BF Blender
Peter Fog (tintwotin) retitled D5511: Video Editor: Move header spacer in to Preview modes from Video Editor: Move header spacer and clean up to Video Editor: Move header spacer in to Preview modes.
Sat, Aug 17, 8:37 AM · BF Blender
Peter Fog (tintwotin) created D5511: Video Editor: Move header spacer in to Preview modes.
Sat, Aug 17, 8:19 AM · BF Blender

Fri, Aug 16

Peter Fog (tintwotin) added a comment to T68690: Cut function in sequencer confused with copy and paste and should be renamed into Split.

+1 for split.

Fri, Aug 16, 12:36 PM · User Interface, Video Sequencer, BF Blender
Peter Fog (tintwotin) added a comment to T68706: Text Editor: file type crash.

Making a Release build, I get no crash. When building with 'make developer' I get the crash.

Fri, Aug 16, 8:31 AM · BF Blender

Thu, Aug 15

Peter Fog (tintwotin) added a comment to T68706: Text Editor: file type crash.

I had some build problems following the Python update, but can build now without errors. It's this build that gives me this error:

Thu, Aug 15, 9:21 PM · BF Blender
Peter Fog (tintwotin) created T68706: Text Editor: file type crash.
Thu, Aug 15, 8:22 PM · BF Blender
Peter Fog (tintwotin) created T68692: Building Blender fails because of 'Missing: Python.h'.
Thu, Aug 15, 2:18 PM · BF Blender
Peter Fog (tintwotin) added a comment to D5474: Horizontal Find & Replace for the Text Editor.

Now this patch has been abandoned, who has a better solution, and how/where to discuss it?

Thu, Aug 15, 11:10 AM · Text Editor, BF Blender

Wed, Aug 14

Valentin (Poulpator) awarded T68656: Info Editor's Replay fails, properly because of bad context a Like token.
Wed, Aug 14, 7:47 PM · BF Blender
Peter Fog (tintwotin) created T68656: Info Editor's Replay fails, properly because of bad context.
Wed, Aug 14, 7:01 PM · BF Blender
Peter Fog (tintwotin) abandoned D5444: Info Editor - Add Menus.
Wed, Aug 14, 12:38 PM · Text Editor, User Interface, BF Blender
Peter Fog (tintwotin) abandoned D5461: Add Toogle Views to Sequencer .
Wed, Aug 14, 12:36 PM · Video Sequencer, BF Blender
Peter Fog (tintwotin) abandoned D5474: Horizontal Find & Replace for the Text Editor.
Wed, Aug 14, 12:35 PM · Text Editor, BF Blender
Peter Fog (tintwotin) added a comment to D5474: Horizontal Find & Replace for the Text Editor.

An alternative way would be if the scrollbar was moved to the right of the side bar and the unused part of the sidebar was made transparent, like the 3D View:

Wed, Aug 14, 8:48 AM · Text Editor, BF Blender

Tue, Aug 13

noki paike (amonpaike) awarded D5474: Horizontal Find & Replace for the Text Editor a Like token.
Tue, Aug 13, 10:58 PM · Text Editor, BF Blender
Peter Fog (tintwotin) added a comment to D5474: Horizontal Find & Replace for the Text Editor.

I think the main question is if this kind of design is worth pursuing? This is more like a test of concept and in the end of the day, it may not the best idea to actually code the operator in Python ex. with no text.get_selection in the API(for adding selection to find box), and the other things @Campbell Barton (campbellbarton) points out here.

Tue, Aug 13, 8:50 PM · Text Editor, BF Blender
Peter Fog (tintwotin) updated the summary of D5474: Horizontal Find & Replace for the Text Editor.
Tue, Aug 13, 7:08 PM · Text Editor, BF Blender
Peter Fog (tintwotin) created D5474: Horizontal Find & Replace for the Text Editor.
Tue, Aug 13, 6:04 PM · Text Editor, BF Blender

Mon, Aug 12

Peter Fog (tintwotin) retitled D5461: Add Toogle Views to Sequencer from Add unexposed operators to Video Editor: Sample Color and Toogle Views. to Add Toogle Views to Sequencer .

Removed Sample Color.

Mon, Aug 12, 7:11 AM · Video Sequencer, BF Blender
Peter Fog (tintwotin) added a comment to T68556: Toggle Maximise Area keyboard shortcut doesn't work in Text Editor despite being listed in the View menu.

I'm sorry - I wasn't clear. I too think is a problem that the maximize short cut key does not work in all areas of Blender, as consistency across all editors is an important thing. So I think autocomplete should have a new short cut key, and maximize should have ctrl+space(everywhere in Blender), but that's not up to me.

Mon, Aug 12, 12:24 AM · BF Blender

Sun, Aug 11

Peter Fog (tintwotin) updated the summary of D5461: Add Toogle Views to Sequencer .
Sun, Aug 11, 11:56 PM · Video Sequencer, BF Blender
Peter Fog (tintwotin) added a comment to D5461: Add Toogle Views to Sequencer .

Changed the operator_context to get Sample Color hotkey to show, however Toggle View only shows hotkey in Sequencer and Sequencer+Preview. I don't know if there is an operator_context which covers all 3 states?

Sun, Aug 11, 11:54 PM · Video Sequencer, BF Blender
Peter Fog (tintwotin) added a comment to T68556: Toggle Maximise Area keyboard shortcut doesn't work in Text Editor despite being listed in the View menu.

If the mouse pointer is on the header, ctrl+space will maximize area. But in the Text Editor area and the Python console area, Ctrl+Space is auto complete.

Sun, Aug 11, 11:36 PM · BF Blender
Peter Fog (tintwotin) retitled D5461: Add Toogle Views to Sequencer from Add unexposed operators to Video Editor: Box Select Handles, Sample Color and Toogle Views. to Add unexposed operators to Video Editor: Sample Color and Toogle Views. .

Updated.

Sun, Aug 11, 11:31 PM · Video Sequencer, BF Blender
Peter Fog (tintwotin) added a comment to T62971: File Browser UI.

Looks great. Is it possible to include a zip filter(+icon)? This would be very useful when installing zipped add-ons.

Sun, Aug 11, 11:19 PM · User Interface, BF Blender
Peter Fog (tintwotin) created D5461: Add Toogle Views to Sequencer .
Sun, Aug 11, 11:04 PM · Video Sequencer, BF Blender
Peter Fog (tintwotin) awarded D5451: Text Editor: Support comment without selection a Party Time token.
Sun, Aug 11, 4:09 PM · BF Blender
Peter Fog (tintwotin) awarded T68512: blender 2.8 malfunctions when reporting an indentation error executing a python script a Like token.
Sun, Aug 11, 8:37 AM · BF Blender

Sat, Aug 10

Peter Fog (tintwotin) awarded rB41f8f08e5188: UI: Add initial context menus for Info and Console editors a Like token.
Sat, Aug 10, 4:26 PM

Fri, Aug 9

noki paike (amonpaike) awarded D5339: Clean up Sequencer/Preview Menu and Preview Menu a Like token.
Fri, Aug 9, 8:37 PM · Video Sequencer, BF Blender
noki paike (amonpaike) awarded D5434: Text Editor Menus - Refreshed & Proposals a Like token.
Fri, Aug 9, 8:30 PM · Text Editor, BF Blender
Peter Fog (tintwotin) added a comment to D5339: Clean up Sequencer/Preview Menu and Preview Menu.

Updated to match current repository.

Fri, Aug 9, 3:28 PM · Video Sequencer, BF Blender
Peter Fog (tintwotin) added a comment to D5434: Text Editor Menus - Refreshed & Proposals.

As Campbell wrote, when he cherry picked what to commit: "would rather separate header reorganization from changing menus", so no header changes, except menus have been committed, so I guess this diff can be closed?

Fri, Aug 9, 2:39 PM · Text Editor, BF Blender
Peter Fog (tintwotin) added a comment to T65723: Sequencer: Invalid timecode values makes strip drawing a mess..

@Richard Antalik (ISS) Could this be added to a VSE To Do list here on developer.blender.org?

Fri, Aug 9, 2:15 PM · Video Sequencer, BF Blender
Peter Fog (tintwotin) added a comment to D5175: Text Editor Missing Hotkey Fix: Comment assigned Ctrl+H, Uncomment assigned Alt+H, Replace Ctrl+R, Find Set Selected Alt + F.

Since the Toggle Comment with a shortcut key was implemented, I'll abandon this diff, though I still think 'Find Set Selected' now called 'Find Next" in the menus, could use a shortcut key.

Fri, Aug 9, 2:11 PM · BF Blender, Text Editor
Peter Fog (tintwotin) updated the summary of D5142: Sequencer: Sidebar Text-Strip Panel adjustments.
Fri, Aug 9, 1:54 PM · Video Sequencer, BF Blender
Peter Fog (tintwotin) added a comment to D5142: Sequencer: Sidebar Text-Strip Panel adjustments.

Shadow now added to check-box-sub-panel:

Fri, Aug 9, 1:54 PM · Video Sequencer, BF Blender
Peter Fog (tintwotin) awarded T68448: Reports & Warnings UI a Like token.
Fri, Aug 9, 12:51 PM · User Interface, BF Blender
Peter Fog (tintwotin) added a comment to D5443: Python Console Menus - Updated .

This update adds the Console menu as Context menu(w/right click).

Fri, Aug 9, 11:23 AM · Text Editor, BF Blender
Peter Fog (tintwotin) updated the summary of D5444: Info Editor - Add Menus.

Update adds the Info menu as Context menu.

Fri, Aug 9, 11:09 AM · Text Editor, User Interface, BF Blender
William Reynish (billreynish) awarded D5444: Info Editor - Add Menus a Like token.
Fri, Aug 9, 10:54 AM · Text Editor, User Interface, BF Blender
Peter Fog (tintwotin) created D5444: Info Editor - Add Menus.
Fri, Aug 9, 10:38 AM · Text Editor, User Interface, BF Blender
Peter Fog (tintwotin) added a comment to D5443: Python Console Menus - Updated .

More class renaming.

Fri, Aug 9, 7:44 AM · Text Editor, BF Blender
Peter Fog (tintwotin) added a comment to D5443: Python Console Menus - Updated .

Cleaned up class naming.

Fri, Aug 9, 7:39 AM · Text Editor, BF Blender
Peter Fog (tintwotin) created D5443: Python Console Menus - Updated .
Fri, Aug 9, 7:31 AM · Text Editor, BF Blender
Peter Fog (tintwotin) awarded rBSadb96478a720: Adding scrollbar default to Python console and Python Info log. a Like token.
Fri, Aug 9, 12:42 AM

Thu, Aug 8

Peter Fog (tintwotin) added a comment to T68325: GPencil: Define the active scene when using the Video Editing Workspace.

One way to solve this is by separating scenes from sequences as suggested in the comments here: T52586 An alternative and simpler approach is to toggle jump to/return from scenes referenced in scene strips. Two add-ons deal with this: https://github.com/tin2tin/change_to_scene_from_scene-strip and https://github.com/snuq/VSESwitchToScene

Thu, Aug 8, 10:20 AM · Grease Pencil, BF Blender
Peter Fog (tintwotin) added a comment to D5434: Text Editor Menus - Refreshed & Proposals.

@Campbell Barton (campbellbarton) Thank you for committing this.

Thu, Aug 8, 10:07 AM · Text Editor, BF Blender
Peter Fog (tintwotin) updated the summary of D5434: Text Editor Menus - Refreshed & Proposals.

Remove Refresh.
Remove wrapping of args.
Rename to Move Line(s) Up/Down.

Thu, Aug 8, 7:21 AM · Text Editor, BF Blender
Peter Fog (tintwotin) added a comment to D5434: Text Editor Menus - Refreshed & Proposals.

Added select Word to Select menu.

Thu, Aug 8, 7:06 AM · Text Editor, BF Blender
Peter Fog (tintwotin) updated the summary of D5434: Text Editor Menus - Refreshed & Proposals.
Thu, Aug 8, 6:45 AM · Text Editor, BF Blender
Peter Fog (tintwotin) added a comment to D5434: Text Editor Menus - Refreshed & Proposals.

Swapped the viewport toggles with the Run Script operator button:

Thu, Aug 8, 6:43 AM · Text Editor, BF Blender

Wed, Aug 7

Peter Fog (tintwotin) added a comment to D5434: Text Editor Menus - Refreshed & Proposals.

It\s possible to move Run Script a little closer, and maybe a little space around the view buttons is an improvement.

Wed, Aug 7, 10:54 PM · Text Editor, BF Blender
Peter Fog (tintwotin) added a comment to D5434: Text Editor Menus - Refreshed & Proposals.

@William Reynish (billreynish) Like this:

Wed, Aug 7, 10:28 PM · Text Editor, BF Blender
William Reynish (billreynish) awarded D5434: Text Editor Menus - Refreshed & Proposals a Like token.
Wed, Aug 7, 10:06 PM · Text Editor, BF Blender
Peter Fog (tintwotin) created D5434: Text Editor Menus - Refreshed & Proposals.
Wed, Aug 7, 10:03 PM · Text Editor, BF Blender
Peter Fog (tintwotin) added a comment to T68366: Text editor slider now blocked.

In the scrollbars with handles this problem has been solved by shortening the scrollbar:

Wed, Aug 7, 3:54 PM · BF Blender

Tue, Aug 6

Peter Fog (tintwotin) added a comment to D5142: Sequencer: Sidebar Text-Strip Panel adjustments.

Changed Shadow to this:

Tue, Aug 6, 3:15 PM · Video Sequencer, BF Blender
Peter Fog (tintwotin) updated the summary of D5339: Clean up Sequencer/Preview Menu and Preview Menu.
Tue, Aug 6, 2:19 PM · Video Sequencer, BF Blender
Peter Fog (tintwotin) updated the summary of D5142: Sequencer: Sidebar Text-Strip Panel adjustments.

Moved "Export Subtitles" into the Sequencer View Menu, and positioned it under the export related render options, since file export buttons do not belong in the Strip Properties Sidebar.

Tue, Aug 6, 2:03 PM · Video Sequencer, BF Blender
Peter Fog (tintwotin) added a comment to rBbb53d2b07a1b: Sequencer: frame offset feature usability.

Checked the dash length again, and now they look fine. No worries.

Tue, Aug 6, 1:35 PM
Peter Fog (tintwotin) added a comment to rBbb53d2b07a1b: Sequencer: frame offset feature usability.

Not too familiar with these functions, but should the Set Overlay Region be disabled when Overlay Type isn't Rectangle?

Tue, Aug 6, 12:34 PM
Peter Fog (tintwotin) added a comment to T66161: Sequencer: Frame overlay marker is almost impossible to see..

Has been fixed by: bb53d2b07a1b

Tue, Aug 6, 12:31 PM · User Interface, Video Sequencer, BF Blender
Peter Fog (tintwotin) created T68299: Blender Preferences > Themes > 'Reset to Default Value' always return the color black.
Tue, Aug 6, 10:17 AM · BF Blender
Peter Fog (tintwotin) added a comment to P1027 Sequencer toolbar.

Oh, this is great. The tools are working. For the first time is is possible to annotate properly in the Sequencer Preview. One thing to keep in mind when desciding on what tools should go in the tool bar, is that hot areas and changing the mouse pointer icon accordingly, can serve as information on current operator, in a way that can keep the clutter out of the toolbar.

Tue, Aug 6, 10:04 AM · Video Sequencer
Peter Fog (tintwotin) added a comment to T67696: Sequencer unable to get resolution, if no preview is open..

If no preview area is open when adding a movie strip to the sequencer, the resolution of that clip is none, but it is actually none until the playhead has been on a frame of that clip, and from there that resolution is remembered.

Tue, Aug 6, 9:40 AM · Video Sequencer, BF Blender

Mon, Aug 5

Peter Fog (tintwotin) added a comment to rB91fa07dfb18f: Text: merge toggle comments into a single operator.

Somehow in the process the comment operator has disappeared from the context menu, and it is missing a tooltip.

Mon, Aug 5, 9:38 AM
Peter Fog (tintwotin) added a comment to rBa2fe386153ee: Text Editor UI.

I realize that it isn't the icon, being moved, but instead it is the scrollbar, which has been shortened, but only the scrollbars with handles in both ends are shortened.

Mon, Aug 5, 9:17 AM

Sun, Aug 4

Peter Fog (tintwotin) added a comment to T54955: Multi-Object Properties Design.

Maybe the selected value could be a little less than white, so a white box would be visible around the value, when the value is only added to the active element.

Sun, Aug 4, 5:48 PM · User Interface, BF Blender: 2.8, Code Quest
Peter Fog (tintwotin) updated subscribers of rB5fde4494f654: Keymap: use Ctrl-/ to toggle comments.

@William Reynish (billreynish) Should Comment and Uncomment be removed now Toggle Comment is added?

Sun, Aug 4, 5:36 PM
Peter Fog (tintwotin) added a comment to rBa2fe386153ee: Text Editor UI.

Noticed that moving the Sidebar to the right, makes the Sidebar icon collide with the scroll bar, so maybe the icon should be moved up above the scrollbar like in ex. the sequencer:

Sun, Aug 4, 5:28 PM
Peter Fog (tintwotin) added a comment to D5028: Text Editor UI updates.

Some parts where rejected when I tried to apply this patch, but has the sidebar been moved to the right in the Workspace too?

Sun, Aug 4, 9:11 AM · BF Blender, Text Editor
Peter Fog (tintwotin) removed a reviewer for D5097: Adding 'Trim Whitespace' and 'Change Case' to Text Editor : William Reynish (billreynish).

Change info about whitespace trimmed lines from popover to report in statusbar.

Sun, Aug 4, 8:34 AM · BF Blender, Python, Text Editor

Sat, Aug 3

Peter Fog (tintwotin) added a comment to T52586: Disable scenes using themselves as sequence strips 2.8 Proposal.

The video editor is fundamentally different from every other editor. Whereas all other editors are used in service of building the current scene

Except the Text Editor. The contents of the Sequencer should be independent of the Scene selection like the Text Editor's contents is.

Sat, Aug 3, 11:10 AM · Video Sequencer, BF Blender: 2.8
Peter Fog (tintwotin) added a comment to T52586: Disable scenes using themselves as sequence strips 2.8 Proposal.

Way back when Motion Builder was Filmbox(more than 20 years ago) is was possible to edit camera mise-en-scenes in a timeline realtime(of the current scene). Since then pretty much all 3D packages and 3D game engines have this feature added, including Unity and Unreal, but still not Blender. With gpencil and the speed of EEVEE, realtime virtual filmmaking(and previz), should be within reach in Blender, but being able to seamless edit cameras(in 3d view) integrated with the sequencer, is a most essential step in that direction. However, without the ability to use cameras of the current scene in the sequencer, this is not possible.

Sat, Aug 3, 9:22 AM · Video Sequencer, BF Blender: 2.8
Peter Fog (tintwotin) added a comment to T52586: Disable scenes using themselves as sequence strips 2.8 Proposal.

We could make the video editor data separate from scenes though (there own data-block for example).

Sat, Aug 3, 7:47 AM · Video Sequencer, BF Blender: 2.8

Fri, Aug 2

Peter Fog (tintwotin) added a comment to T66112: Sequencer Annotation Tool(in the Sequencer Timeline Sidebar) does not draw and has not D-key assigned..

Resolved by D5306

Fri, Aug 2, 10:36 PM · Video Sequencer, BF Blender
Peter Fog (tintwotin) added a comment to D5339: Clean up Sequencer/Preview Menu and Preview Menu.

Renamed "Show Image Metadata" back to "Show Metadata" because apparently it also extracts metadata from Matroska video files.

Fri, Aug 2, 8:41 AM · Video Sequencer, BF Blender
Peter Fog (tintwotin) added a comment to D5028: Text Editor UI updates.

Both suggestions are fine by me. What do @William Reynish (billreynish) say?

Fri, Aug 2, 12:14 AM · BF Blender, Text Editor

Wed, Jul 31

Peter Fog (tintwotin) updated the summary of D5097: Adding 'Trim Whitespace' and 'Change Case' to Text Editor .
Wed, Jul 31, 8:38 PM · BF Blender, Python, Text Editor
Peter Fog (tintwotin) retitled D5097: Adding 'Trim Whitespace' and 'Change Case' to Text Editor from Adding 'Trim Whitespaces', 'Split & Join Line' and 'Change Case' to Text Editor to Adding 'Trim Whitespace' and 'Change Case' to Text Editor .

Updated the diff: Using clipboard code has been removed, instead code to get selection by @Dalai Felinto (dfelinto) has been used: https://developer.blender.org/diffusion/BAC/browse/master/text_editor_hastebin.py If/when this function will be added to the API this def can be removed.

Wed, Jul 31, 1:10 PM · BF Blender, Python, Text Editor
Peter Fog (tintwotin) added a comment to D5028: Text Editor UI updates.

Here are the small changes I mentioned above:


Unfortunately I can't update this patch(missing rights?), but here it is for download:

Wed, Jul 31, 9:59 AM · BF Blender, Text Editor
Peter Fog (tintwotin) added a comment to D5341: Add missing Fractional Zoom shortcut keys for Sequencer Preview.

@Richard Antalik (ISS) Multi Cam shortcuts keys are not affected by Numpad keys.

Wed, Jul 31, 9:47 AM · Video Sequencer, BF Blender
Peter Fog (tintwotin) awarded D5375: UI Demonstration: File Manager Thumbnail View a Like token.
Wed, Jul 31, 7:44 AM · User Interface

Tue, Jul 30

Peter Fog (tintwotin) added a comment to D5028: Text Editor UI updates.

Oh, this is strange. I did a build with this patch, and now the Text Editor won't load a text block into the text editor anymore.
Can anyone confirm this?

Tue, Jul 30, 2:00 PM · BF Blender, Text Editor
Peter Fog (tintwotin) added a comment to D5028: Text Editor UI updates.

On the icon, the first button seems to copy&paste&find, which I feel isn't covered with an Import icon or a Paste icon, but maybe that's just me? (Actually this function should have been build into the Find operator - so highlighting text and hit ctrl+f would do that, and there would be no need for a button). Maybe the Eyedropper icon could cover the process of extracting a string and add it to the panel? Maybe it's better, but it still doesn't cover the additional find function.

Tue, Jul 30, 12:28 PM · BF Blender, Text Editor
Peter Fog (tintwotin) added a comment to D5028: Text Editor UI updates.

I'm sorry. I'm hesitating a bit, because are we sure about the 'Download' icons? Or do we need more exact icons created for these buttons?
Are we sure, we want the Find&Replace settings above the search fields(normally these settings are below the search fields)?
Should Find&Replace be on top in the Sidebar. Having the most used functions on top?
Are we sure we do not want Highlight Line unselected as default setting?

Tue, Jul 30, 11:35 AM · BF Blender, Text Editor
Peter Fog (tintwotin) added a comment to D5175: Text Editor Missing Hotkey Fix: Comment assigned Ctrl+H, Uncomment assigned Alt+H, Replace Ctrl+R, Find Set Selected Alt + F.

If Ctrl + / is it, which is industry standart for Line Comment(?), then this is all it takes(in Python) to add Line Comment to the current comment function. Code by Lapineige:

class BetterComment(bpy.types.Operator):
    """  """
    bl_idname = "text.better_comment"
    bl_label = "Better Comment"
Tue, Jul 30, 11:27 AM · BF Blender, Text Editor
Peter Fog (tintwotin) added a comment to D5175: Text Editor Missing Hotkey Fix: Comment assigned Ctrl+H, Uncomment assigned Alt+H, Replace Ctrl+R, Find Set Selected Alt + F.

Ctrl + / is actually line comment(in PyCharm), where the location of the cursor determines which line to comment and this is typically a toggle function - this line comment function doesn't exist in Blender.

Tue, Jul 30, 10:18 AM · BF Blender, Text Editor
Peter Fog (tintwotin) added a comment to D5175: Text Editor Missing Hotkey Fix: Comment assigned Ctrl+H, Uncomment assigned Alt+H, Replace Ctrl+R, Find Set Selected Alt + F.

Sure I can change it to Ctrl + /, but as mentioned, that keyboard combination does not work on non-English keyboards:

A common shortcut for this is Ctrl + / but this doesn't work on non English keyboards(Ctrl + Shift + 7(=Ctrl + /) doesn't work in Blender)

T66607 is another example of shortcut keys not working on non-English keyboards. Unless the plan is to start adding localized shortcut keymaps, I think we should be careful not to utilize keys which do not work on non-English keyboards.

Tue, Jul 30, 8:05 AM · BF Blender, Text Editor