Milestone 1105050722 #127

Merged
Dandielo merged 78 commits from milestone-1105050722 into master 2022-07-10 21:32:15 +00:00
Dandielo commented 2022-06-26 11:40:57 +00:00 (Migrated from github.com)

This is the final issues of the "Font and UI" milestone. Any additional major changes are going to be addressed here.

This is the final issues of the "Font and UI" milestone. Any additional major changes are going to be addressed here.
Dandielo (Migrated from github.com) reviewed 2022-06-28 08:39:13 +00:00
@ -48,6 +48,26 @@
"executable": "%VULKAN_SDK%/Bin/glslc.exe",
"options": [ "--target-env=vulkan", "-fshader-stage=frag", "-o", "sprite-pix.spv", "sprite-pix.glsl" ]
},
{
Dandielo (Migrated from github.com) commented 2022-06-28 08:39:13 +00:00

Planned improvements to remove all the code duplication. #128

Planned improvements to remove all the code duplication. #128
Dandielo (Migrated from github.com) reviewed 2022-06-28 08:45:49 +00:00
@ -22,6 +22,9 @@ namespace ice
using uptr = std::uintptr_t;
using usize = ice::u64;
Dandielo (Migrated from github.com) commented 2022-06-28 08:45:49 +00:00

A quick change preceding a bigger refactor: #129

A quick change preceding a bigger refactor: #129
Dandielo (Migrated from github.com) reviewed 2022-06-28 08:47:14 +00:00
Dandielo (Migrated from github.com) commented 2022-06-28 08:47:14 +00:00

Remove 'Sleep' call from task_thead_v2::run

Remove 'Sleep' call from `task_thead_v2::run`
Dandielo (Migrated from github.com) reviewed 2022-06-28 08:49:32 +00:00
@ -49,0 +60,4 @@
ice::stringid_hint(name)
);
// TODO: To be refactored with the planned introduction of a proper 'size' type.
Dandielo (Migrated from github.com) commented 2022-06-28 08:49:32 +00:00

Part of #129

Part of #129
Dandielo (Migrated from github.com) reviewed 2022-06-28 08:53:08 +00:00
Dandielo (Migrated from github.com) commented 2022-06-28 08:53:08 +00:00

Part of #130

Part of #130
Dandielo (Migrated from github.com) reviewed 2022-06-28 08:54:52 +00:00
@ -50,1 +50,4 @@
bool requested_exit() const noexcept override
{
return ice::shards::contains(_runner->previous_frame().shards(), "action/game/exit"_shard);
Dandielo (Migrated from github.com) commented 2022-06-28 08:54:52 +00:00

Requires documentation in code and on the wiki pages.

Requires documentation in code and on the wiki pages.
Dandielo (Migrated from github.com) reviewed 2022-06-28 08:56:08 +00:00
@ -75,3 +94,6 @@ namespace ice
template<>
Dandielo (Migrated from github.com) commented 2022-06-28 08:56:08 +00:00

Might want to start separating some parts of this file.

Might want to start separating some parts of this file.
Dandielo (Migrated from github.com) reviewed 2022-06-28 08:58:01 +00:00
Dandielo (Migrated from github.com) reviewed 2022-06-28 08:58:24 +00:00
Dandielo (Migrated from github.com) commented 2022-06-28 08:58:23 +00:00

Remove all the unnecessary comments.

Remove all the unnecessary comments.
Dandielo (Migrated from github.com) reviewed 2022-06-28 09:01:53 +00:00
@ -178,6 +178,29 @@ namespace ice
return object_ptr;
Dandielo (Migrated from github.com) commented 2022-06-28 09:01:53 +00:00

Part of: #129

Part of: #129
Dandielo (Migrated from github.com) reviewed 2022-06-28 09:04:58 +00:00
@ -0,0 +1,7 @@
#include <ice/font.hxx>
Dandielo (Migrated from github.com) commented 2022-06-28 09:04:58 +00:00

Remove this file?

Remove this file?
Dandielo (Migrated from github.com) reviewed 2022-06-28 09:07:05 +00:00
Dandielo (Migrated from github.com) commented 2022-06-28 09:07:04 +00:00

Add a in-code comment how this works.

Add a in-code comment how this works.
Dandielo (Migrated from github.com) reviewed 2022-06-28 09:07:54 +00:00
@ -0,0 +1,12 @@
#pragma once
Dandielo (Migrated from github.com) commented 2022-06-28 09:07:53 +00:00

Remove this file?

Remove this file?
Dandielo (Migrated from github.com) reviewed 2022-06-28 09:08:31 +00:00
@ -0,0 +1,25 @@
#pragma once
Dandielo (Migrated from github.com) commented 2022-06-28 09:08:30 +00:00

Remove this type?

Remove this type?
Dandielo (Migrated from github.com) reviewed 2022-06-28 09:09:05 +00:00
@ -0,0 +44,4 @@
ice::Span<ice::ui::UIResourceData const> resources
) noexcept -> ice::ui::UpdateResult;
static constexpr ice::ui::UpdateStage Constant_UpdateStageOrder[]{
Dandielo (Migrated from github.com) commented 2022-06-28 09:09:04 +00:00

This is not usable, ale actual logic is more complicated.

This is not usable, ale actual logic is more complicated.
Dandielo (Migrated from github.com) reviewed 2022-06-28 09:09:20 +00:00
@ -0,0 +5,4 @@
namespace ice::ui
{
struct DrawData
Dandielo (Migrated from github.com) commented 2022-06-28 09:09:20 +00:00

Unused?

Unused?
Dandielo (Migrated from github.com) reviewed 2022-06-28 09:10:01 +00:00
@ -0,0 +1,30 @@
#pragma once
Dandielo (Migrated from github.com) commented 2022-06-28 09:10:00 +00:00

Clean up this file.

Clean up this file.
Dandielo (Migrated from github.com) reviewed 2022-06-28 09:10:20 +00:00
@ -0,0 +1,88 @@
#pragma once
Dandielo (Migrated from github.com) commented 2022-06-28 09:10:20 +00:00

Add some static_assert tests.

Add some static_assert tests.
Dandielo (Migrated from github.com) reviewed 2022-06-28 09:12:26 +00:00
Dandielo (Migrated from github.com) commented 2022-06-28 09:12:26 +00:00

This could be replaced with ice::shards::transform_all, but need double checking.

This could be replaced with `ice::shards::transform_all`, but need double checking.
Dandielo (Migrated from github.com) reviewed 2022-06-28 09:13:02 +00:00
@ -0,0 +1,6 @@
{
"asset": {
Dandielo (Migrated from github.com) commented 2022-06-28 09:13:02 +00:00

Remove this file.

Remove this file.
Dandielo (Migrated from github.com) reviewed 2022-07-03 22:43:48 +00:00
@ -0,0 +37,4 @@
Font* gfx_font = reinterpret_cast<Font*>(data);
FontAtlas* gfx_atlas = reinterpret_cast<FontAtlas*>(
ice::memory::ptr_align_forward(gfx_font + 1, alignof(FontAtlas))
); // TODO: Multi-atlas support
Dandielo (Migrated from github.com) commented 2022-07-03 22:43:47 +00:00

Create a task for font-atlas support and another for multiple atlasses in one font asset.

Create a task for font-atlas support and another for multiple atlasses in one font asset.
Dandielo (Migrated from github.com) reviewed 2022-07-03 22:44:42 +00:00
@ -0,0 +124,4 @@
msdf_atlas::FontGeometry geometry{ &glyphs };
msdf_atlas::Charset charset = msdf_atlas::Charset::ASCII;
// TODO: Properly support bigger charsets.
Dandielo (Migrated from github.com) commented 2022-07-03 22:44:42 +00:00

Create a issues for handling big utf8 character sets.

Create a issues for handling big utf8 character sets.
Dandielo (Migrated from github.com) reviewed 2022-07-03 22:46:46 +00:00
@ -0,0 +1,356 @@
#include "ip_ui_oven_elements.hxx"
Dandielo (Migrated from github.com) commented 2022-07-03 22:46:46 +00:00

Might need to change this one into a follow up issue.

Might need to change this one into a follow up issue.
Dandielo (Migrated from github.com) reviewed 2022-07-03 22:47:14 +00:00
@ -0,0 +133,4 @@
//ice::usize const type_end = str.find_first_of(u8' ');
//ice::Utf8String const data_type = str.substr(0, type_end);
//if (out_action.action_type == ice::ui::ActionType::Shard
Dandielo (Migrated from github.com) commented 2022-07-03 22:47:14 +00:00

This might be a follow up issues, as currently there is no use case to have this solved.

This might be a follow up issues, as currently there is no use case to have this solved.
Dandielo (Migrated from github.com) reviewed 2022-07-03 22:48:54 +00:00
@ -0,0 +1,367 @@
#include <ice/ui_element.hxx>
Dandielo (Migrated from github.com) commented 2022-07-03 22:48:53 +00:00

Might be a bug?

Might be a bug?
Dandielo (Migrated from github.com) reviewed 2022-07-03 22:55:32 +00:00
Dandielo (Migrated from github.com) commented 2022-07-03 22:55:31 +00:00

This does not work, as prev_frame is immutable, and we cannot mutate shards.
Bu we might add a new api to move and transform shards from one array to another.

This does not work, as prev_frame is immutable, and we cannot mutate shards. Bu we might add a new api to move and transform shards from one array to another.
Dandielo (Migrated from github.com) reviewed 2022-07-03 22:56:15 +00:00
Dandielo (Migrated from github.com) left a comment

A few more changes required but we are comming to an end of this milestone!

A few more changes required but we are comming to an end of this milestone!
Dandielo (Migrated from github.com) reviewed 2022-07-03 22:56:44 +00:00
@ -0,0 +1,6 @@
{
"asset": {
Dandielo (Migrated from github.com) commented 2022-07-03 22:56:44 +00:00

Or not ;d

Or not ;d
Dandielo (Migrated from github.com) reviewed 2022-07-04 21:40:10 +00:00
@ -0,0 +37,4 @@
Font* gfx_font = reinterpret_cast<Font*>(data);
FontAtlas* gfx_atlas = reinterpret_cast<FontAtlas*>(
ice::memory::ptr_align_forward(gfx_font + 1, alignof(FontAtlas))
); // TODO: Multi-atlas support
Dandielo (Migrated from github.com) commented 2022-07-04 21:40:10 +00:00

Created #132

Created #132
Dandielo (Migrated from github.com) reviewed 2022-07-04 21:42:51 +00:00
@ -0,0 +124,4 @@
msdf_atlas::FontGeometry geometry{ &glyphs };
msdf_atlas::Charset charset = msdf_atlas::Charset::ASCII;
// TODO: Properly support bigger charsets.
Dandielo (Migrated from github.com) commented 2022-07-04 21:42:50 +00:00

Create #133

Create #133
Dandielo (Migrated from github.com) reviewed 2022-07-04 21:43:51 +00:00
@ -0,0 +1,356 @@
#include "ip_ui_oven_elements.hxx"
Dandielo (Migrated from github.com) commented 2022-07-04 21:43:51 +00:00

For now we removed this code, as we might not need it for a longer time. If that's the case we would anyway need to re-visit most of the code.

For now we removed this code, as we might not need it for a longer time. If that's the case we would anyway need to re-visit most of the code.
Dandielo (Migrated from github.com) reviewed 2022-07-04 21:47:03 +00:00
@ -0,0 +133,4 @@
//ice::usize const type_end = str.find_first_of(u8' ');
//ice::Utf8String const data_type = str.substr(0, type_end);
//if (out_action.action_type == ice::ui::ActionType::Shard
Dandielo (Migrated from github.com) commented 2022-07-04 21:47:03 +00:00

For now there is no known situation where we would need to send additional data with shards.
This issues might need a revisit at a later point. #134

For now there is no known situation where we would need to send additional data with shards. This issues might need a revisit at a later point. #134
Dandielo (Migrated from github.com) reviewed 2022-07-04 21:50:29 +00:00
@ -0,0 +1,367 @@
#include <ice/ui_element.hxx>
Dandielo (Migrated from github.com) commented 2022-07-04 21:50:29 +00:00

For now this will be not addressed. As there is no need for the anchoring feature.

For now this will be not addressed. As there is no need for the anchoring feature.
Dandielo (Migrated from github.com) reviewed 2022-07-04 21:54:13 +00:00
Dandielo (Migrated from github.com) left a comment

We are approaching the end of this milestone. Just missing colors to have an MVP at this point.

We are approaching the end of this milestone. Just missing colors to have an MVP at this point.
Dandielo (Migrated from github.com) reviewed 2022-07-06 21:51:14 +00:00
Dandielo (Migrated from github.com) left a comment

Features finished.

Features finished.
Dandielo (Migrated from github.com) reviewed 2022-07-07 22:23:28 +00:00
Dandielo (Migrated from github.com) left a comment

Last remaining thing are fixes on Linux.

Last remaining thing are fixes on Linux.
Dandielo (Migrated from github.com) reviewed 2022-07-08 00:22:38 +00:00
Dandielo (Migrated from github.com) commented 2022-07-08 00:22:38 +00:00

We might keep it for now, and extend the API to have access to simmilar functions across platforms.

We might keep it for now, and extend the API to have access to simmilar functions across platforms.
Dandielo commented 2022-07-10 20:59:45 +00:00 (Migrated from github.com)
Added minimal documentation for fonts and UI: * https://github.com/iceshard-engine/engine/wiki/Docs-Fonts * https://github.com/iceshard-engine/engine/wiki/Docs-UI
Sign in to join this conversation.
No description provided.