1
0
mirror of synced 2024-11-28 09:30:51 +01:00

fix: Rendering issues with Hex Editor Minimap

This commit is contained in:
WerWolv 2024-01-29 23:28:53 +01:00
parent 76e304c34e
commit 18b717594f
2 changed files with 7 additions and 9 deletions

View File

@ -500,7 +500,7 @@ namespace hex::plugin::builtin {
} }
// Right click menu // Right click menu
if (ImGui::IsMouseReleased(ImGuiMouseButton_Right) && ImGui::IsWindowHovered(ImGuiHoveredFlags_ChildWindows)) if (ImGui::IsMouseReleased(ImGuiMouseButton_Right) && ImGui::IsWindowHovered(ImGuiHoveredFlags_ChildWindows) && !ImGui::IsAnyItemHovered())
RequestOpenPopup::post("hex.builtin.menu.edit"); RequestOpenPopup::post("hex.builtin.menu.edit");
} }

View File

@ -193,10 +193,6 @@ namespace hex::ui {
} }
void HexEditor::drawMinimap(ImVec2 characterSize) { void HexEditor::drawMinimap(ImVec2 characterSize) {
if (const auto &visualizers = ContentRegistry::HexEditor::impl::getMiniMapVisualizers(); m_miniMapVisualizer == nullptr && !visualizers.empty())
m_miniMapVisualizer = visualizers.front();
ImS64 numRows = m_provider == nullptr ? 0 : (m_provider->getSize() / m_bytesPerRow) + ((m_provider->getSize() % m_bytesPerRow) == 0 ? 0 : 1); ImS64 numRows = m_provider == nullptr ? 0 : (m_provider->getSize() / m_bytesPerRow) + ((m_provider->getSize() % m_bytesPerRow) == 0 ? 0 : 1);
auto window = ImGui::GetCurrentWindowRead(); auto window = ImGui::GetCurrentWindowRead();
@ -211,9 +207,9 @@ namespace hex::ui {
constexpr static u64 RowCount = 256; constexpr static u64 RowCount = 256;
const auto rowHeight = innerRect.GetSize().y / RowCount; const auto rowHeight = innerRect.GetSize().y / RowCount;
const auto scrollPos = m_scrollPosition.get(); const ImS64 scrollPos = m_scrollPosition.get();
const auto grabSize = rowHeight * m_visibleRowCount; const auto grabSize = rowHeight * m_visibleRowCount;
const auto grabPos = (RowCount - m_visibleRowCount) * (double(scrollPos) / numRows); const ImS64 grabPos = (RowCount - m_visibleRowCount) * (double(scrollPos) / numRows);
auto drawList = ImGui::GetWindowDrawList(); auto drawList = ImGui::GetWindowDrawList();
@ -241,8 +237,8 @@ namespace hex::ui {
drawList->ChannelsSetCurrent(0); drawList->ChannelsSetCurrent(0);
std::vector<u8> rowData(m_bytesPerRow); std::vector<u8> rowData(m_bytesPerRow);
const auto drawStart = scrollPos - grabPos; const auto drawStart = std::max<ImS64>(0, scrollPos - grabPos);
for (u64 y = drawStart; y < std::min<u64>(drawStart + RowCount, m_provider->getSize() / m_bytesPerRow); y += 1) { for (ImS64 y = drawStart; y < std::min<ImS64>(drawStart + RowCount, m_provider->getSize() / m_bytesPerRow); y += 1) {
const auto rowStart = bb.Min + ImVec2(0, (y - drawStart) * rowHeight); const auto rowStart = bb.Min + ImVec2(0, (y - drawStart) * rowHeight);
const auto rowEnd = rowStart + ImVec2(bb.GetSize().x, rowHeight); const auto rowEnd = rowStart + ImVec2(bb.GetSize().x, rowHeight);
@ -407,6 +403,8 @@ namespace hex::ui {
m_currDataVisualizer = visualizer; m_currDataVisualizer = visualizer;
return; return;
} }
if (const auto &visualizers = ContentRegistry::HexEditor::impl::getMiniMapVisualizers(); m_miniMapVisualizer == nullptr && !visualizers.empty())
m_miniMapVisualizer = visualizers.front();
const auto bytesPerCell = m_currDataVisualizer->getBytesPerCell(); const auto bytesPerCell = m_currDataVisualizer->getBytesPerCell();
const u16 columnCount = m_bytesPerRow / bytesPerCell; const u16 columnCount = m_bytesPerRow / bytesPerCell;