v6.0.0

.github/workflows/build.yaml

@@ -104,7 +104,7 @@ jobs:
           --gcov-executable '${{ matrix.gcov_executable }}';
 
       - name: Windows - Test and coverage
-        if: runner.os == 'Windows'
+        if: runner.os == 'Windows' && false
         working-directory: ./build
         run: >
           OpenCppCoverage.exe

CHANGELOG.md

@@ -1,34 +1,8 @@
 Changelog
 =========
 
-Development
------------
-### Component
-- Bugfix: Fix a crash with ResizeableSplit. See #1023.
-  - Clamp screen size to terminal size.
-  - Disallow `ResizeableSplit` with negative size.
-
-### Dom
-- Bugfix: Disallow specifying a negative size constraint. See #1023.
-
-
-6.0.2 (2025-03-30)
------
-
-### Component
-- BugFix: Fix major crash on Windows affecting all components. See #1020
-- BugFix: Fix focusRelative.
-
-6.0.1 (2025-03-28)
------
-
-Same as v6.0.0.
-
-Due to a problem tag v6.0.0 was replaced. This isn't a good practice and affect
-developers that started using it in the short timeframe. Submitting a new
-release with the same content is the best way to fix this.
-
-See #1017 and #1019.
+current (development) 
+---------------------
 
 6.0.0 (2025-03-23)
 -----
@@ -100,10 +74,6 @@ See #1017 and #1019.
   - See `selectionStyleReset` decorator.
 - Breaking change: Change how "focus"/"select" are handled. This fixes the
   behavior.
-- Breaking change: `Component::OnRender()` becomes the method to override to
-  render a component. This replaces `Component::Render()` that is still in use
-  to call the rendering method on the children. This change allows to fix a
-  couple of issues around focus handling.
 
 ### Screen
 - Feature: Add `Box::IsEmpty()`.

CMakeLists.txt

@@ -2,7 +2,7 @@ cmake_minimum_required(VERSION 3.12)
 
 project(ftxui
   LANGUAGES CXX
-  VERSION 6.0.2
+  VERSION 6.0.0
   DESCRIPTION "C++ Functional Terminal User Interface."
 )
 

README.md

@@ -372,7 +372,7 @@ include(FetchContent)
 
 FetchContent_Declare(ftxui
   GIT_REPOSITORY https://github.com/ArthurSonzogni/ftxui
-  GIT_TAG v6.0.2
+  GIT_TAG v6.0.0
 )
 
 FetchContent_GetProperties(ftxui)
@@ -388,7 +388,7 @@ If you don't, FTXUI may be used from the following packages:
 - [conan.io](https://conan.io/center/ftxui)
 - [openSUSE](https://build.opensuse.org/package/show/devel:libraries:c_c++/ftxui)
 - 
-[![Packaging status](https://repology.org/badge/vertical-allrepos/libftxui.svg)](https://repology.org/project/libftxui/versions)
+[![Packaging status](https://repology.org/badge/vertical-allrepos/ftxui.svg)](https://repology.org/project/ftxui/versions)
 
 If you choose to build and link FTXUI yourself, `ftxui-component` must be first in the linking order relative to the other FTXUI libraries, i.e.
 ```bash

src/ftxui/component/resizable_split.cpp

@@ -77,16 +77,16 @@ class ResizableSplitBase : public ComponentBase {
 
     switch (options_->direction()) {
       case Direction::Left:
-        options_->main_size() = std::max(0, event.mouse().x - box_.x_min);
+        options_->main_size() = event.mouse().x - box_.x_min;
         return true;
       case Direction::Right:
-        options_->main_size() = std::max(0, box_.x_max - event.mouse().x);
+        options_->main_size() = box_.x_max - event.mouse().x;
         return true;
       case Direction::Up:
-        options_->main_size() = std::max(0, event.mouse().y - box_.y_min);
+        options_->main_size() = event.mouse().y - box_.y_min;
         return true;
       case Direction::Down:
-        options_->main_size() = std::max(0, box_.y_max - event.mouse().y);
+        options_->main_size() = box_.y_max - event.mouse().y;
         return true;
     }
 

src/ftxui/component/screen_interactive.cpp

@@ -34,7 +34,6 @@
 #include "ftxui/dom/requirement.hpp"                  // for Requirement
 #include "ftxui/screen/pixel.hpp"                     // for Pixel
 #include "ftxui/screen/terminal.hpp"                  // for Dimensions, Size
-#include "ftxui/screen/util.hpp"                      // for util::clamp
 
 #if defined(_WIN32)
 #define DEFINE_CONSOLEV2_PROPERTIES
@@ -918,15 +917,15 @@ void ScreenInteractive::Draw(Component component) {
       break;
     case Dimension::TerminalOutput:
       dimx = terminal.dimx;
-      dimy = util::clamp(document->requirement().min_y, 0, terminal.dimy);
+      dimy = document->requirement().min_y;
       break;
     case Dimension::Fullscreen:
       dimx = terminal.dimx;
       dimy = terminal.dimy;
       break;
     case Dimension::FitComponent:
-      dimx = util::clamp(document->requirement().min_x, 0, terminal.dimx);
-      dimy = util::clamp(document->requirement().min_y, 0, terminal.dimy);
+      dimx = std::min(document->requirement().min_x, terminal.dimx);
+      dimy = std::min(document->requirement().min_y, terminal.dimy);
       break;
   }
 

src/ftxui/dom/focus.cpp

@@ -36,7 +36,7 @@ Decorator focusPositionRelative(float x, float y) {
 
     void ComputeRequirement() override {
       NodeDecorator::ComputeRequirement();
-      requirement_.focused.enabled = true;
+      requirement_.focused.enabled = false;
       requirement_.focused.node = this;
       requirement_.focused.box.x_min = int(float(requirement_.min_x) * x_);
       requirement_.focused.box.y_min = int(float(requirement_.min_y) * y_);

src/ftxui/dom/node.cpp

@@ -125,27 +125,27 @@ void Render(Screen& screen, Node* node, Selection& selection) {
     node->Select(selection);
   }
 
+  // Setting the cursor to the right position allow folks using CJK (China,
+  // Japanese, Korean, ...) characters to see their [input method editor]
+  // displayed at the right location. See [issue].
+  //
+  // [input method editor]:
+  // https://en.wikipedia.org/wiki/Input_method
+  //
+  // [issue]:
+  // https://github.com/ArthurSonzogni/FTXUI/issues/2#issuecomment-505282355
+  //
+  // Unfortunately, Microsoft terminal do not handle properly hiding the
+  // cursor. Instead the character under the cursor is hidden, which is a big
+  // problem. As a result, we can't enable setting cursor to the right
+  // location. It will be displayed at the bottom right corner.
+  // See:
+  // https://github.com/microsoft/terminal/issues/1203
+  // https://github.com/microsoft/terminal/issues/3093
   if (node->requirement().focused.enabled
 #if defined(FTXUI_MICROSOFT_TERMINAL_FALLBACK)
-      // Setting the cursor to the right position allow folks using CJK (China,
-      // Japanese, Korean, ...) characters to see their [input method editor]
-      // displayed at the right location. See [issue].
-      //
-      // [input method editor]:
-      // https://en.wikipedia.org/wiki/Input_method
-      //
-      // [issue]:
-      // https://github.com/ArthurSonzogni/FTXUI/issues/2#issuecomment-505282355
-      //
-      // Unfortunately, Microsoft terminal do not handle properly hiding the
-      // cursor. Instead the character under the cursor is hidden, which is a
-      // big problem. As a result, we can't enable setting cursor to the right
-      // location. It will be displayed at the bottom right corner.
-      // See:
-      // https://github.com/microsoft/terminal/issues/1203
-      // https://github.com/microsoft/terminal/issues/3093
-      &&
-      node->requirement().focused.cursor_shape != Screen::Cursor::Shape::Hidden
+      ||
+      node->requirement().focused.cursor_shape == Screen::Cursor::Shape::Hidden
 #endif
   ) {
     screen.SetCursor(Screen::Cursor{

src/ftxui/dom/size.cpp

@@ -19,7 +19,7 @@ class Size : public Node {
       : Node(unpack(std::move(child))),
         direction_(direction),
         constraint_(constraint),
-        value_(std::max(0, value)) {}
+        value_(value) {}
 
   void ComputeRequirement() override {
     Node::ComputeRequirement();

src/ftxui/screen/terminal.cpp

@@ -48,25 +48,12 @@ Dimensions& FallbackSize() {
   return g_fallback_size;
 }
 
-bool Contains(const std::string& s, const char* key) {
-  return s.find(key) != std::string::npos;
+const char* Safe(const char* c) {
+  return (c != nullptr) ? c : "";
 }
 
-// https://github.com/gabime/spdlog/blob/885b5473e291833b148eeac3b7ce227e582cd88b/include/spdlog/details/os-inl.h#L566
-std::string getenv_safe(const char *field) {
-#if defined(_MSC_VER)
-#if defined(__cplusplus_winrt)
-  return std::string{};  // not supported under uwp
-#else
-  size_t len = 0;
-  char buf[1024];
-  bool ok = ::getenv_s(&len, buf, sizeof(buf), field) == 0;
-  return ok ? buf : std::string{};
-#endif
-#else  // revert to getenv
-  char *buf = ::getenv(field); // NOLINT(*-mt-unsafe)
-  return buf ? buf : std::string{};
-#endif
+bool Contains(const std::string& s, const char* key) {
+  return s.find(key) != std::string::npos;
 }
 
 Terminal::Color ComputeColorSupport() {
@@ -74,12 +61,12 @@ Terminal::Color ComputeColorSupport() {
   return Terminal::Color::TrueColor;
 #endif
 
-  std::string COLORTERM = getenv_safe("COLORTERM");
+  std::string COLORTERM = Safe(std::getenv("COLORTERM"));  // NOLINT
   if (Contains(COLORTERM, "24bit") || Contains(COLORTERM, "truecolor")) {
     return Terminal::Color::TrueColor;
   }
 
-  std::string TERM = getenv_safe("TERM");
+  std::string TERM = Safe(std::getenv("TERM"));  // NOLINT
   if (Contains(COLORTERM, "256") || Contains(TERM, "256")) {
     return Terminal::Color::Palette256;
   }