Fixup docs

Author: karnkaul

guide/src/shader_objects/drawing_triangle.md

@@ -3,13 +3,16 @@
 Add a `ShaderProgram` to `App` and its create function:
 
 ```cpp
+[[nodiscard]] auto asset_path(std::string_view uri) const -> fs::path;
+
+// ...
 void create_shader();
 
 // ...
 std::optional<ShaderProgram> m_shader{};
 ```
 
-Implement and call `create_shader()`:
+Implement and call `create_shader()` (and `asset_path()`):
 
 ```cpp
 void App::create_shader() {
@@ -22,6 +25,10 @@ void App::create_shader() {
   };
   m_shader.emplace(shader_ci);
 }
+
+auto App::asset_path(std::string_view const uri) const -> fs::path {
+  return m_assets_dir / uri;
+}
 ```
 
 Before `render()` grows to an unwieldy size, extract the higher level logic into two member functions:

guide/src/shader_objects/shader_program.md

@@ -1,6 +1,6 @@
 # Shader Program
 
-SPIR-V modules are binary files with a stride/alignment of 4 bytes. The Vulkan API accepts a span of `std::uint32_t`s, so we need to load it into such a buffer (and _not_ `std::vector<std::byte>` or other 1-byte equivalents). We will encapsulate both vertex and fragment shaders into a single `ShaderProgram`, which will also bind the shaders before a draw, and expose/set various dynamic states.
+We will encapsulate both vertex and fragment shaders into a single `ShaderProgram`, which will also bind the shaders before a draw, and expose/set various dynamic states.
 
 In `shader_program.hpp`, first add a `ShaderProgramCreateInfo` struct:
 
@@ -233,35 +233,3 @@ void ShaderProgram::bind_shaders(vk::CommandBuffer const command_buffer) const {
   command_buffer.bindShadersEXT(stages_v, shaders);
 }
 ```
-
-## TODO: MOVE
-
-Add new members to `App`:
-
-```cpp
-void create_pipelines();
-
-[[nodiscard]] auto asset_path(std::string_view uri) const -> fs::path;
-```
-
-Add code to load shaders in `create_pipelines()` and call it before starting the main loop:
-
-```cpp
-void App::create_pipelines() {
-  auto shader_loader = ShaderLoader{*m_device};
-  // we only need shader modules to create the pipelines, thus no need to
-  // store them as members.
-  auto const vertex = shader_loader.load(asset_path("shader.vert"));
-  auto const fragment = shader_loader.load(asset_path("shader.frag"));
-  if (!vertex || !fragment) {
-    throw std::runtime_error{"Failed to load Shaders"};
-  }
-  std::println("[lvk] Shaders loaded");
-
-  // TODO
-}
-
-auto App::asset_path(std::string_view const uri) const -> fs::path {
-  return m_assets_dir / uri;
-}
-```