Add font styling

[RobLoach]

• Add a demo showing how to set the fonts.

[StrikerTheHedgefox]

Is this something you'd consider merging?

[RobLoach]

Would be good to have one of the demo's showing an example of it.

[StrikerTheHedgefox]

How does one compile the demo? I can try to make one.

[RobLoach]

They use cmake currently...

mkdir build
cd build
cmake ..
make

[StrikerTheHedgefox]

Trying to make my changes in nuklear_console_demo.c, but it's not letting me access most of the nuklear functions...

In file included from D:/sourcecode/nuklear_console-fonts/demo/raylib/main.c:12:
D:/sourcecode/nuklear_console-fonts/demo/common/nuklear_console_demo.c: In function 'nuklear_console_demo_init':
D:/sourcecode/nuklear_console-fonts/demo/common/nuklear_console_demo.c:111:20: error: implicit declaration of function 'nk_font_atlas_add_from_file' [-Wimplicit-function-declaration]
  111 |         demofont = nk_font_atlas_add_from_file(atlas, "./crillee.otf", 14, 0);
      |                    ^~~~~~~~~~~~~~~~~~~~~~~~~~~

Also, raylib is the only type cmake was able to successfully set up, I couldn't get SDL or GLFW builds working.

EDIT: Got it working for raylib:

Only thing, is the element doesn't adjust size for the font.

[StrikerTheHedgefox]

It would appear the SDL demo is broken:

rm -rf nuklear_console_demo_sdl
cc main.c -std=c99 -pedantic -O0 `sdl2-config --cflags` -o nuklear_console_demo_sdl
In file included from main.c:24:
../../vendor/Nuklear/demo/sdl_renderer/nuklear_sdl_renderer.h: In function 'nk_sdl_render':
../../vendor/Nuklear/demo/sdl_renderer/nuklear_sdl_renderer.h:120:9: error: implicit declaration of function 'NK_MEMSET' [-Wimplicit-function-declaration]
  120 |         NK_MEMSET(&config, 0, sizeof(config));
      |         ^~~~~~~~~
In file included from ../common/nuklear_console_demo.c:7,
                 from main.c:29:
../common/../../vendor/nuklear_gamepad/nuklear_gamepad.h: In function 'nk_gamepad_init_with_source':
../common/../../vendor/nuklear_gamepad/nuklear_gamepad.h:319:5: error: implicit declaration of function 'nk_zero' [-Wimplicit-function-declaration]
  319 |     nk_zero(gamepads, sizeof(struct nk_gamepads));
      |     ^~~~~~~
../common/../../vendor/nuklear_gamepad/nuklear_gamepad.h:336:9: error: implicit declaration of function 'nk_itoa'; did you mean '_itoa'? [-Wimplicit-function-declaration]
  336 |         nk_itoa(&gamepads->gamepads[i].name[j], (long)(i + 1));
      |         ^~~~~~~
      |         _itoa
In file included from ../common/nuklear_console_demo.c:10:
../common/../../nuklear_console_button.h: In function 'nk_console_button':
../common/../../nuklear_console.h:235:27: error: implicit declaration of function 'nk_malloc'; did you mean 'SDL_malloc'? [-Wimplicit-function-declaration]
  235 | #define NK_CONSOLE_MALLOC nk_malloc
      |                           ^~~~~~~~~
../common/../../nuklear_console_button.h:182:61: note: in expansion of macro 'NK_CONSOLE_MALLOC'
  182 |     nk_console_button_data* data = (nk_console_button_data*)NK_CONSOLE_MALLOC(nk_handle_id(0), NULL, sizeof(nk_console_button_data));
      |                                                             ^~~~~~~~~~~~~~~~~
In file included from ../common/../../nuklear_console.h:284:
../common/../../nuklear_console_button.h:182:36: warning: cast to pointer from integer of different size [-Wint-to-pointer-cast]
  182 |     nk_console_button_data* data = (nk_console_button_data*)NK_CONSOLE_MALLOC(nk_handle_id(0), NULL, sizeof(nk_console_button_data));
      |                                    ^
In file included from ../common/../../nuklear_console.h:285:
../common/../../nuklear_console_checkbox.h: In function 'nk_console_checkbox':
../common/../../nuklear_console_checkbox.h:116:5: error: implicit declaration of function 'NK_ASSERT' [-Wimplicit-function-declaration]
  116 |     NK_ASSERT(active != NULL);
      |     ^~~~~~~~~
../common/../../nuklear_console_checkbox.h:117:38: warning: cast to pointer from integer of different size [-Wint-to-pointer-cast]
  117 |     nk_console_checkbox_data* data = (nk_console_checkbox_data*)NK_CONSOLE_MALLOC(nk_handle_id(0), NULL, sizeof(nk_console_checkbox_data));
      |                                      ^
In file included from ../common/../../nuklear_console.h:286:
../common/../../nuklear_console_color.h: In function 'nk_console_color':
../common/../../nuklear_console_color.h:116:35: warning: cast to pointer from integer of different size [-Wint-to-pointer-cast]
  116 |     nk_console_color_data* data = (nk_console_color_data*)NK_CONSOLE_MALLOC(nk_handle_id(0), NULL, sizeof(nk_console_color_data));
      |                                   ^
In file included from ../common/../../nuklear_console.h:287:
../common/../../nuklear_console_combobox.h: In function 'nk_console_combobox':
../common/../../nuklear_console_combobox.h:92:38: warning: cast to pointer from integer of different size [-Wint-to-pointer-cast]
   92 |     nk_console_combobox_data* data = (nk_console_combobox_data*)NK_CONSOLE_MALLOC(nk_handle_id(0), NULL, sizeof(nk_console_combobox_data));
      |                                      ^
In file included from ../common/../../nuklear_console.h:288:
../common/../../nuklear_console_file.h: In function 'nk_console_file_entry_onclick':
../common/../../nuklear_console_file.h:232:5: error: implicit declaration of function 'NK_MEMCPY'; did you mean 'HAVE_MEMCPY'? [-Wimplicit-function-declaration]
  232 |     NK_MEMCPY(data->directory + len, (void*)button->label, (nk_size)(nk_strlen(button->label) + 1));
      |     ^~~~~~~~~
      |     HAVE_MEMCPY
../common/../../nuklear_console_file.h: In function 'nk_console_file_add_entry':
../common/../../nuklear_console_file.h:285:21: warning: cast to pointer from integer of different size [-Wint-to-pointer-cast]
  285 |     button->label = (const char*)NK_CONSOLE_MALLOC(nk_handle_id(0), NULL, (nk_size)(sizeof(char)) * (nk_size)(len + 1));
      |                     ^
../common/../../nuklear_console_file.h: In function 'nk_console_file':
../common/../../nuklear_console_file.h:421:34: warning: cast to pointer from integer of different size [-Wint-to-pointer-cast]
  421 |     nk_console_file_data* data = (nk_console_file_data*)NK_CONSOLE_MALLOC(nk_handle_id(0), NULL, sizeof(nk_console_file_data));
      |                                  ^
In file included from ../common/../../nuklear_console.h:290:
../common/../../nuklear_console_image.h: In function 'nk_console_image_color':
../common/../../nuklear_console_image.h:111:9: warning: cast to pointer from integer of different size [-Wint-to-pointer-cast]
  111 |         (nk_console_image_data*)NK_CONSOLE_MALLOC(nk_handle_id(0), NULL, sizeof(*data));
      |         ^
In file included from ../common/../../nuklear_console.h:291:
../common/../../nuklear_console_input.h: In function 'nk_console_input':
../common/../../nuklear_console_input.h:235:35: warning: cast to pointer from integer of different size [-Wint-to-pointer-cast]
  235 |     nk_console_input_data* data = (nk_console_input_data*)NK_CONSOLE_MALLOC(nk_handle_id(0), NULL, sizeof(nk_console_input_data));
      |                                   ^
In file included from ../common/../../nuklear_console.h:294:
../common/../../nuklear_console_progress.h: In function 'nk_console_progress':
../common/../../nuklear_console_progress.h:35:38: warning: cast to pointer from integer of different size [-Wint-to-pointer-cast]
   35 |     nk_console_progress_data* data = (nk_console_progress_data*)NK_CONSOLE_MALLOC(nk_handle_id(0), NULL, sizeof(nk_console_progress_data));
      |                                      ^
In file included from ../common/../../nuklear_console.h:295:
../common/../../nuklear_console_property.h: In function 'nk_console_property_int':
../common/../../nuklear_console_property.h:206:38: warning: cast to pointer from integer of different size [-Wint-to-pointer-cast]
  206 |     nk_console_property_data* data = (nk_console_property_data*)NK_CONSOLE_MALLOC(nk_handle_id(0), NULL, sizeof(nk_console_property_data));
      |                                      ^
In file included from ../common/../../nuklear_console.h:296:
../common/../../nuklear_console_radio.h: In function 'nk_console_radio':
../common/../../nuklear_console_radio.h:227:35: warning: cast to pointer from integer of different size [-Wint-to-pointer-cast]
  227 |     nk_console_radio_data* data = (nk_console_radio_data*)NK_CONSOLE_MALLOC(nk_handle_id(0), NULL, sizeof(nk_console_radio_data));
      |                                   ^
In file included from ../common/../../nuklear_console.h:297:
../common/../../nuklear_console_row.h: In function 'nk_console_row_begin':
../common/../../nuklear_console_row.h:109:33: warning: cast to pointer from integer of different size [-Wint-to-pointer-cast]
  109 |     nk_console_row_data* data = (nk_console_row_data*)NK_CONSOLE_MALLOC(nk_handle_id(0), NULL, sizeof(nk_console_row_data));
      |                                 ^
In file included from ../common/../../nuklear_console.h:299:
../common/../../nuklear_console_textedit.h: In function 'nk_console_textedit':
../common/../../nuklear_console_textedit.h:349:38: warning: cast to pointer from integer of different size [-Wint-to-pointer-cast]
  349 |     nk_console_textedit_data* data = (nk_console_textedit_data*)NK_CONSOLE_MALLOC(nk_handle_id(0), NULL, sizeof(nk_console_textedit_data));
      |                                      ^
../common/../../nuklear_console.h: In function 'nk_console_malloc':
../common/../../nuklear_console.h:235:27: error: returning 'int' from a function with return type 'void *' makes pointer from integer without a cast -Wint-conversion]
  235 | #define NK_CONSOLE_MALLOC nk_malloc
      |                           ^
../common/../../nuklear_console.h:791:12: note: in expansion of macro 'NK_CONSOLE_MALLOC'
  791 |     return NK_CONSOLE_MALLOC(unused, old, size);
      |            ^~~~~~~~~~~~~~~~~
../common/../../nuklear_console.h: In function 'nk_console_mfree':
../common/../../nuklear_console.h:244:25: error: implicit declaration of function 'nk_mfree'; did you mean 'nk_free'? [-Wimplicit-function-declaratio]
  244 | #define NK_CONSOLE_FREE nk_mfree
      |                         ^~~~~~~~
../common/../../nuklear_console.h:796:5: note: in expansion of macro 'NK_CONSOLE_FREE'
  796 |     NK_CONSOLE_FREE(unused, ptr);
      |     ^~~~~~~~~~~~~~~
make[1]: *** [Makefile:30: nuklear_console_demo_sdl] Error 1
make[1]: Leaving directory '/d/sourcecode/nuklear_console-fonts/demo/sdl_renderer'
make: *** [Makefile:19: sdl] Error 2

[StrikerTheHedgefox]

I think being able to define a full style per-element might be the better way to go here. This way we could color elements like labels and whatnot, set fonts, size, etc.

[RobLoach]

That's a great idea. Would be nice to be able to customize on a per-component level. Not sure how much more memory that would eat up, but would give a lot of control.

[StrikerTheHedgefox]

That's a great idea. Would be nice to be able to customize on a per-component level. Not sure how much more memory that would eat up, but would give a lot of control.

I imagine if they're just pointers to styles, they could be re-used, and it'd just take up as much memory as is required for the number of styles defined, rather than the number of elements.