summaryrefslogtreecommitdiffstats
path: root/editor/plugins/sprite_frames_editor_plugin.cpp
Commit message (Collapse)AuthorAgeFilesLines
...
* | Use compile-time Unicode string conversionHugo Locurcio2023-08-071-1/+1
|/ | | | | Thanks to this syntax introduced in C++11, this reduces the amount of work that needs to be performed at run-time while making the code more terse.
* Merge pull request #79743 from TokageItLab/fix-name-list-aspYuri Sizov2023-07-241-6/+5
|\ | | | | | | Make animation name list scroll to new animation in `SpriteEditor`
| * Make animation name list scroll to new animation in SpriteEditorSilc Lizard (Tokage) Renew2023-07-211-6/+5
| | | | | | | | Co-authored-by: DarellLdark <darell.ldark@gmail.com>
* | Fix rename animation in SpriteFramesEditor/AnimationNodeStateMachineEditorRindbee2023-07-181-1/+4
|/ | | | | | | | | | | | When the name suffix grows, the old name is used if it is obtained first. Fix the case where the following error message would appear when renaming an animation. ``` ERROR: Animation '' doesn't exist. at: get_frame_count (scene/resources/sprite_frames.cpp:71) ```
* Extract and reorganize texture resource classesHendrik Brucker2023-07-141-0/+1
|
* Merge pull request #74341 from dalexeev/sprite-frames-texture-filterRémi Verschelde2023-05-221-0/+2
|\ | | | | | | Use nearest with mipmaps texture filter in SpriteFrames editor plugin
| * Use nearest with mipmaps texture filter in SpriteFrames editor pluginDanil Alexeev2023-03-041-0/+2
| |
* | Hide Animation Frames section when there are no animations and show messageHakim2023-05-191-5/+21
| |
* | Make SpriteFrames editor toolbar a FlowContainerkleonc2023-05-131-22/+36
| |
* | Improve includes of EditorNode (and everything else)Yuri Sizov2023-04-071-1/+1
| | | | | | | | | | | | Also start organizing editor-specific GUI components into a dedicated folder, `editor/gui`. Also move `editor_file_server` next to the rest of debugger classes.
* | Improve SpriteFrameEditor frame addition orderingNinni Pipping2023-03-301-95/+316
|/
* Set default name if SpriteFrames name is emptyEldor Zang2023-02-171-1/+5
|
* Fix ghost SpriteFramesEditor cause crashSilc Renew2023-02-061-17/+14
|
* Fix switching method in SpriteFramesEditorSilc Renew2023-01-311-20/+18
|
* Few improvements for SpriteFrames EditorDanil Alexeev2023-01-301-25/+32
|
* Make AnimatedSprite's playback API consistent with AnimationPlayerSilc Renew2023-01-261-102/+398
|
* Merge pull request #48570 from Calinou/animation-editors-new-nameRémi Verschelde2023-01-201-2/+2
|\ | | | | | | Tweak the name for new animations in the editor
| * Tweak the name for new animations in the editorHugo Locurcio2022-07-131-2/+2
| | | | | | | | | | | | | | | | | | | | - Use snake_case to "suggest" the naming that fits the Godot style guide. - Fully spell out "new_animation" since both editors can fit it in full. - Don't internationalize the new animation name to have consistent behavior between both editors. - Since the recommendation is to follow snake_case, special characters should also be avoided in the name to make animations easier to refer to in code.
* | Merge pull request #71379 from KoBeWi/destruction_of_compatibility_functionRémi Verschelde2023-01-181-4/+1
|\ \ | | | | | | Remove set_drag_forwarding_compat()
| * | Remove set_drag_forwarding_compat()kobewi2023-01-141-4/+1
| | |
* | | Add EditorUndoRedoManager singletonkobewi2023-01-161-15/+15
|/ /
* | Add expand modes to TextureRectkobewi2023-01-121-1/+1
| |
* | Change set_drag_forwarding() to use callables.Juan Linietsky2023-01-101-1/+1
| | | | | | | | | | | | | | | | * This solution is much cleaner than the one in 3.x thanks to the use of callables. * Works without issues in any language (no need to worry about camel or snake case). * Editor code uses a compatibility function (too much work to redo). Fixes #59899
* | Use BitField<> in core type masksJuan Linietsky2023-01-081-2/+2
| | | | | | | | | | | | | | | | * All core types masks are now correctly marked as bitfields. * The enum hacks in MouseButtonMask and many other types are gone. This ensures that binders to other languages non C++ can actually implement type safe bitmasks. * Most bitmask operations replaced by functions in BitField<> * Key is still a problem because its enum and mask at the same time. While it kind of works in C++, this most likely can't be implemented safely in other languages and will have to be changed at some point. Mostly left as-is. * Documentation and API dump updated to reflect bitfields in core types.
* | Merge pull request #65609 from dalexeev/animated-spriteRémi Verschelde2023-01-051-89/+166
|\ \ | | | | | | | | | `AnimatedSprite{2D,3D}` improvements
| * | `AnimatedSprite{2D,3D}` improvementsDanil Alexeev2023-01-051-89/+166
| | | | | | | | | | | | | | | * Add support for individual frame duration to `SpriteFrames`. * Various minor improvements.
* | | One Copyright Update to rule them allRémi Verschelde2023-01-051-29/+29
|/ / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | As many open source projects have started doing it, we're removing the current year from the copyright notice, so that we don't need to bump it every year. It seems like only the first year of publication is technically relevant for copyright notices, and even that seems to be something that many companies stopped listing altogether (in a version controlled codebase, the commits are a much better source of date of publication than a hardcoded copyright statement). We also now list Godot Engine contributors first as we're collectively the current maintainers of the project, and we clarify that the "exclusive" copyright of the co-founders covers the timespan before opensourcing (their further contributions are included as part of Godot Engine contributors). Also fixed "cf." Frenchism - it's meant as "refer to / see".
* | Use forward-declarations in big editor classestrollodel2022-11-291-0/+2
| |
* | Cleanup remaining EditorUndoRedoManager usageskobewi2022-11-241-6/+14
| |
* | Fix misc formatting issuesRémi Verschelde2022-11-141-1/+1
| | | | | | | | | | | | | | Somehow I missed the failing CI report from trailing whitespace. Fixed a couple typos found by codespell while at it, and misc trailing semicolons in one of the files.
* | `SpriteFramesEditor` Fix calculating frame index from mouse positionkleonc2022-10-301-3/+7
| |
* | Merge pull request #66337 from EricEzaM/sprite-framesRémi Verschelde2022-10-131-42/+63
|\ \ | | | | | | | | | Ensure control built-in shortcuts are matched exactly & add shortcuts for SpriteFrames editor
| * | Add shortcuts for SpriteFrames editor. Small renaming changes for clarity & ↵Eric M2022-10-131-42/+63
| | | | | | | | | | | | consistency.
* | | Fix MSVC warnings, rename shadowed variables, fix uninitialized values, ↵bruvzg2022-10-071-2/+2
|/ / | | | | | | change warnings=all to use /W4.
* | Merge pull request #65421 from V-Sekai/spriteframes_read_onlyRémi Verschelde2022-09-141-1/+30
|\ \
| * | Add read only mode to SpriteFrames editor.SaracenOne2022-09-061-1/+30
| | |
* | | Improve naming of theme properties throughout GUI codeYuri Sizov2022-09-061-1/+1
|/ / | | | | | | | | | | | | | | | | | | | | | | | | | | Rename ItemList's bg -> panel Rename ItemList's bg_focus -> focus Rename ProgressBar's bg -> background Rename ProgressBar's fg -> fill Rename Tree's bg -> panel Rename Tree's bg_focus -> focus Rename ScrollContainer's bg -> panel Rename FileDialog's *_icon_modulate -> *_icon_color Rename FileDialog's files_disabled -> file_disabled_color Rename CheckButton's on/off -> checked/unchecked Rename check_v_adjust -> check_v_offset
* | Rename `CanvasItem.update()` to `queue_redraw()`Micky2022-08-291-4/+4
| | | | | | | | | | | | | | | | | | | | Affects a lot of classes. Very thoroughly checked signal connections and deferred calls to this method, add_do_method/add_undo_method calls, and so on. Also renames the internal `_update_callback()` to `_redraw_callback()` for consistency. Just a few comments have also been changed to say "redraw". In CPUParticles2D, there was a private variable with the same name. It has been renamed to `do_redraw`.
* | Revert "Remove NOTIFICATION_ENTER_TREE when paired with ↵Rémi Verschelde2022-08-291-0/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | NOTIFICATION_THEME_CHANGED" This reverts commit 4b817a565cab8af648c88cfc7ab6481e86ee3625. Fixes #64988. Fixes #64997. This caused several regressions (#64988, #64997, https://github.com/godotengine/godot/issues/64997#issuecomment-1229970605) which point at a flaw in the current logic: - `Control::NOTIFICATION_ENTER_TREE` triggers a *deferred* notification with `NOTIFCATION_THEME_CHANGED` as introduced in #62845. - Some classes use their `THEME_CHANGED` to cache theme items in member variables (e.g. `style_normal`, etc.), and use those member variables in `ENTER_TREE`, `READY`, `DRAW`, etc. Since the `THEME_CHANGE` notification is now deferred, they end up accessing invalid state and this can lead to not applying theme properly (e.g. for EditorHelp) or crashing (e.g. for EditorLog or CodeEdit). So we need to go back to the drawing board and see if `THEME_CHANGED` can be called earlier so that the previous logic still works? Or can we refactor all engine code to make sure that: - `ENTER_TREE` and similar do not depend on theme properties cached in member variables. - Or `THEME_CHANGE` does trigger a general UI update to make sure that any bad theme handling in `ENTER_TREE` and co. gets fixed when `THEME_CHANGE` does arrive for the first time. But that means having a temporary invalid (and possibly still crashing) state, and doing some computations twice which might be heavy (e.g. `EditorHelp::_update_doc()`).
* | Merge pull request #64885 from Mickeon/rename-tooltip-hintRémi Verschelde2022-08-281-17/+17
|\ \ | | | | | | Rename `hint_tooltip` to `tooltip_text` & setter getter
| * | Rename `hint_tooltip` to `tooltip_text` & setgetMicky2022-08-271-17/+17
| | | | | | | | | | | | | | | | | | | | | | | | `hint_tooltip` -> `tooltip_text` `set_tooltip` -> `set_tooltip_text` `_get_tooltip` -> `get_tooltip_text` Updates documentation, too.
* | | Remove NOTIFICATION_ENTER_TREE when paired with NOTIFICATION_THEME_CHANGEDAaron Record2022-08-271-1/+0
|/ /
* | Add per-scene UndoRedokobewi2022-08-221-1/+6
| |
* | Remove Signal connect bindsJuan Linietsky2022-07-291-9/+9
| | | | | | | | | | Remove the optional argument p_binds from `Object::connect` since it was deprecated by Callable.bind(). Changed all uses of it to Callable.bind()
* | Capitalize the SpriteFrames search bar placeholderFireForge2022-07-181-1/+1
| | | | | | | | Follow-up to https://github.com/godotengine/godot/pull/49488
* | SpriteFramesEditor Add animation searchboxkleonc2022-07-181-4/+24
|/
* Add ok_button_text to AcceptDialog and cancel_button_text to ConfirmationDialogFireForge2022-07-091-4/+4
|
* Use range iterators for RBSet in most casesAaron Record2022-05-191-4/+4
|
* Replace most uses of Map by HashMapreduz2022-05-161-2/+2
| | | | | | | | | | | | * Map is unnecessary and inefficient in almost every case. * Replaced by the new HashMap. * Renamed Map to RBMap and Set to RBSet for cases that still make sense (order matters) but use is discouraged. There were very few cases where replacing by HashMap was undesired because keeping the key order was intended. I tried to keep those (as RBMap) as much as possible, but might have missed some. Review appreciated!
* Remove `RES` and `REF` typedefs in favor of spelled out `Ref<>`Hugo Locurcio2022-05-031-3/+3
| | | | | | These typedefs don't save much typing compared to the full `Ref<Resource>` and `Ref<RefCounted>`, yet they sometimes introduce confusion among new contributors.