Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

タブ幅が半角単位8桁を超える場合でも下線が最後まで描画されるようにする #1673

Merged
1 commit merged into from May 23, 2021
Merged

タブ幅が半角単位8桁を超える場合でも下線が最後まで描画されるようにする #1673

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
30 changes: 15 additions & 15 deletions sakura_core/view/figures/CFigure_Comma.cpp
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -28,10 +28,8 @@
#include "types/CTypeSupport.h"
#include "apiwrap/StdApi.h"

void _DispTab( CGraphics& gr, DispPos* pDispPos, const CEditView* pcView );

// -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- //
// CFigure_Comma //
// CFigure_Comma //
Comment on lines -31 to +32
Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

CFigure_Tab.cppをコピーした名残だと思われます。
未定義かつ使用されることはないと思うので消してしまいます。

// -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- //

bool CFigure_Comma::Match(const wchar_t* pText, int nTextLen) const
Expand All @@ -58,51 +56,53 @@ void CFigure_Comma::DispSpace(CGraphics& gr, DispPos* pDispPos, CEditView* pcVie
DispPos& sPos=*pDispPos;

//必要なインターフェース
const CLayoutMgr& pcLayoutMgr = pcView->GetDocument()->m_cLayoutMgr;
Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

CLayoutMgrを編集可能な状態でアクセスする必要はないので、const参照経由で使用します。

const CTextMetrics* pMetrics=&pcView->GetTextMetrics();
const CTextArea* pArea=&pcView->GetTextArea();

int nLineHeight = pMetrics->GetHankakuDy();
int nCharWidth = pMetrics->GetCharPxWidth(); // Layout→Px
Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

換算方法の見直しで使用されなくなるので除去
（CLayoutMgr由来のピクセル単位数値は、CLayoutMgrの持つ基準値で換算すべき。）


CTypeSupport cTabType(pcView,COLORIDX_TAB);

// これから描画するタブ幅
CLayoutXInt tabDispWidthLayout = pcView->m_pcEditDoc->m_cLayoutMgr.GetActualTsvSpace( sPos.GetDrawCol(), L',' );
int tabDispWidth = (Int)tabDispWidthLayout;
CLayoutXInt nTabLayoutWidth = pcLayoutMgr.GetActualTsvSpace(sPos.GetDrawCol(), L',');
size_t nTabDispWidth = static_cast<Int>(nTabLayoutWidth) / pcLayoutMgr.GetWidthPerKeta();
Copy link
Author

@ghost ghost May 22, 2021 •
edited by ghost
Loading

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

変数名変更
nTabLayoutWidht：タブのレイアウト幅（ピクセル単位数値）
nTabDispWidth：タブの表示幅（半角単位桁数）
nTabDispWidhtは後でsize_t型の値と演算するのでsize_tで宣言します。

if( pcView->m_bMiniMap ){
CLayoutMgr mgrTemp;
mgrTemp.SetTabSpaceInfo(pcView->m_pcEditDoc->m_cLayoutMgr.GetTabSpaceKetas(),
CLayoutXInt(pcView->GetTextMetrics().GetHankakuWidth()) );
tabDispWidthLayout = mgrTemp.GetActualTabSpace(sPos.GetDrawCol());
tabDispWidth = (Int)tabDispWidthLayout;
mgrTemp.SetTabSpaceInfo(pcLayoutMgr.GetTabSpaceKetas(), static_cast<CLayoutXInt>(pMetrics->GetHankakuWidth()));
nTabLayoutWidth = mgrTemp.GetActualTabSpace(sPos.GetDrawCol());
nTabDispWidth = static_cast<Int>(nTabLayoutWidth) / mgrTemp.GetWidthPerKeta();
}

// タブ記号領域
RECT rcClip2;
rcClip2.left = sPos.GetDrawPos().x;
rcClip2.right = rcClip2.left + nCharWidth * tabDispWidth;
rcClip2.right = rcClip2.left + static_cast<Int>(nTabLayoutWidth);
Comment on lines -83 to +80
Copy link
Author

@ghost ghost May 22, 2021 •
edited by ghost
Loading

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

変更前はタブのピクセル幅に文字のピクセル幅を乗じていました。
（この計算結果は事実上「桁数×ピクセル単位文字幅^2」になってしまいます。）

Copy link
Author

@ghost ghost May 22, 2021 •
edited by ghost
Loading

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

見直したら2乗にならないことがわかりました。

sakura/sakura_core/view/CTextMetrics.h

Line 60 in 6b4f0a8

[[nodiscard]] int GetCharPxWidth() const { return 1; }

ただ、tabDispWidthの中身はピクセル単位数値なのだから、この演算をする意味はないと思います。

if( rcClip2.left < pArea->GetAreaLeft() ){
rcClip2.left = pArea->GetAreaLeft();
}
rcClip2.top = sPos.GetDrawPos().y;
rcClip2.bottom = sPos.GetDrawPos().y + nLineHeight;
int nLen = wcslen( m_pTypeData->m_szTabViewString );
Copy link
Author

@ghost ghost May 22, 2021 •
edited by ghost
Loading

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Code Smell：未使用変数の定義・縮小型変換


if( pArea->IsRectIntersected(rcClip2) ){
if( cTabType.IsDisp() ){ //CSVモード
std::wstring szViewString = L",";
if (szViewString.length() < nTabDispWidth) {
szViewString.append(nTabDispWidth - szViewString.length(), L' ');
}
::ExtTextOut(
gr,
sPos.GetDrawPos().x,
sPos.GetDrawPos().y,
ExtTextOutOption() & ~(bTrans? ETO_OPAQUE: 0),
&rcClip2,
L", ",
tabDispWidth <= 8 ? tabDispWidth : 8, // Sep. 22, 2002 genta
szViewString.c_str(),
static_cast<UINT>(szViewString.length()),
pMetrics->GetDxArray_AllHankaku()
);
}
}

//Xを進める
sPos.ForwardDrawCol(tabDispWidthLayout);
sPos.ForwardDrawCol(nTabLayoutWidth);
}
36 changes: 19 additions & 17 deletions sakura_core/view/figures/CFigure_Tab.cpp
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -61,39 +61,41 @@ void CFigure_Tab::DispSpace(CGraphics& gr, DispPos* pDispPos, CEditView* pcView,
DispPos& sPos=*pDispPos;

//必要なインターフェース
const CLayoutMgr& pcLayoutMgr = pcView->GetDocument()->m_cLayoutMgr;
const CTextMetrics* pMetrics=&pcView->GetTextMetrics();
const CTextArea* pArea=&pcView->GetTextArea();

int nLineHeight = pMetrics->GetHankakuDy();
int nCharWidth = pMetrics->GetCharPxWidth(); // Layout→Px

CTypeSupport cTabType(pcView,COLORIDX_TAB);

// これから描画するタブ幅
CLayoutXInt tabDispWidthLayout = pcView->m_pcEditDoc->m_cLayoutMgr.GetActualTsvSpace( sPos.GetDrawCol(), WCODE::TAB );
int tabDispWidth = (Int)tabDispWidthLayout;
CLayoutXInt nTabLayoutWidth = pcLayoutMgr.GetActualTsvSpace(sPos.GetDrawCol(), WCODE::TAB);
size_t nTabDispWidth = static_cast<Int>(nTabLayoutWidth) / pcLayoutMgr.GetWidthPerKeta();
if( pcView->m_bMiniMap ){
CLayoutMgr mgrTemp;
mgrTemp.SetTabSpaceInfo(pcView->m_pcEditDoc->m_cLayoutMgr.GetTabSpaceKetas(),
CLayoutXInt(pcView->GetTextMetrics().GetHankakuWidth()) );
tabDispWidthLayout = mgrTemp.GetActualTabSpace(sPos.GetDrawCol());
tabDispWidth = (Int)tabDispWidthLayout;
mgrTemp.SetTabSpaceInfo(pcLayoutMgr.GetTabSpaceKetas(), static_cast<CLayoutXInt>(pMetrics->GetHankakuWidth()));
nTabLayoutWidth = mgrTemp.GetActualTabSpace(sPos.GetDrawCol());
nTabDispWidth = static_cast<Int>(nTabLayoutWidth) / mgrTemp.GetWidthPerKeta();
}

// タブ記号領域
RECT rcClip2;
rcClip2.left = sPos.GetDrawPos().x;
rcClip2.right = rcClip2.left + nCharWidth * tabDispWidth;
rcClip2.right = rcClip2.left + static_cast<Int>(nTabLayoutWidth);
if( rcClip2.left < pArea->GetAreaLeft() ){
rcClip2.left = pArea->GetAreaLeft();
}
rcClip2.top = sPos.GetDrawPos().y;
rcClip2.bottom = sPos.GetDrawPos().y + nLineHeight;
int nLen = wcslen( m_pTypeData->m_szTabViewString );

if( pArea->IsRectIntersected(rcClip2) ){
if( cTabType.IsDisp() && TABARROW_STRING == m_pTypeData->m_bTabArrow ){ //タブ通常表示 //@@@ 2003.03.26 MIK
int fontNo = WCODE::GetFontNo(m_pTypeData->m_szTabViewString[0]);
std::wstring szForeViewString = m_pTypeData->m_szTabViewString;
if (szForeViewString.length() < nTabDispWidth) {
szForeViewString.append(nTabDispWidth - szForeViewString.length(), L' ');
}
int fontNo = WCODE::GetFontNo(szForeViewString[0]);
if( fontNo ){
SFONT sFont;
sFont.m_sFontAttr = gr.GetCurrentMyFontAttr();
Expand All @@ -108,24 +110,24 @@ void CFigure_Tab::DispSpace(CGraphics& gr, DispPos* pDispPos, CEditView* pcView,
sPos.GetDrawPos().y + nHeightMargin,
ExtTextOutOption() & ~(bTrans? ETO_OPAQUE: 0),
&rcClip2,
m_pTypeData->m_szTabViewString,
// tabDispWidth <= 8 ? tabDispWidth : 8, // Sep. 22, 2002 genta
nLen,
szForeViewString.c_str(),
static_cast<UINT>(szForeViewString.length()),
pMetrics->GetDxArray_AllHankaku() // FIXME:半角固定？
Copy link
Author

@ghost ghost May 21, 2021 •
edited by ghost
Loading

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ここでは半角文字の文字間隔配列を取得していますが、タブ記号には全角文字も設定できるので、
GenerateDxArray2()の方が良いのかもしれません。
ただ、全角文字を指定しても現状正しく表示されているように見えるので、実害があるのか判断できませんでした。

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

指摘ではありませんが、タブ幅の最小値って2なんでしたっけ？

最小値に1以下を指定できるとしたら、この修正内容と関係なく問題ありな気がします。

Copy link
Author

@ghost ghost May 22, 2021 •
edited by ghost
Loading

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

おそらく設定できます。
タイプ別ダイアログのコードを見たところ、スピンコントロールに対して「正の整数で1～64の範囲内」という入力規則が設定されていました。

sakura/sakura_core/typeprop/CPropTypesScreen.cpp

Lines 412 to 418 in 0843eb9

if( nVal < 1 ){
nVal = 1;
}
if( nVal > 64 ){
nVal = 64;
}
::SetDlgItemInt( hwndDlg, IDC_EDIT_TABSPACE, nVal, FALSE );

タブ幅を1にしている人がいないとも限らないので、GenerateDxArray2を使用するように仕様変更（＝このPRには含めない）しましょうか？

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

めんどくさそうな感じっすねｗ

TAB幅に1を指定するのは趣味だと思うので、放置しましょう。

TAB幅に1を指定してあるのなら、
タブ描画に使うカスタム文字に全角文字を指定できてはいけない気がします。

TAB幅2未満のときだけ全角文字を無効とする、ないし、TAB描画文字に全角を指定したときは幅を2未満にできないようにする、みたいな修正を入れておくのがベターだと思います。
（自己責任っちゅうことで、放置でよいような・・・）

);
if( fontNo ){
gr.PopMyFont();
}
}else{
//背景
std::wstring szBackViewString(nTabDispWidth, L' ');
::ExtTextOut(
gr,
sPos.GetDrawPos().x,
sPos.GetDrawPos().y,
ExtTextOutOption() & ~(bTrans? ETO_OPAQUE: 0),
&rcClip2,
L" ",
8,
szBackViewString.c_str(),
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

不具合修正として直すべき箇所はココだけのように見えました。

static_cast<UINT>(szBackViewString.length()),
pMetrics->GetDxArray_AllHankaku()
);

Expand Down Expand Up @@ -154,7 +156,7 @@ void CFigure_Tab::DispSpace(CGraphics& gr, DispPos* pDispPos, CEditView* pcView,
gr,
nPosLeft,
sPos.GetDrawPos().y,
nCharWidth * tabDispWidth - (nPosLeft - sPos.GetDrawPos().x), // Tab Area一杯に 2013/4/11 Uchi
static_cast<Int>(nTabLayoutWidth) - (nPosLeft - sPos.GetDrawPos().x), // Tab Area一杯に 2013/4/11 Uchi
Comment on lines -157 to +159
Copy link
Author

@ghost ghost May 22, 2021 •
edited by ghost
Loading

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ここもタブのピクセル幅に文字のピクセル幅を乗じていました。

pMetrics->GetHankakuHeight(),
gr.GetCurrentMyFontBold() || m_pTypeData->m_ColorInfoArr[COLORIDX_TAB].m_sFontAttr.m_bBoldFont,
gr.GetCurrentTextForeColor()
Expand All @@ -165,7 +167,7 @@ void CFigure_Tab::DispSpace(CGraphics& gr, DispPos* pDispPos, CEditView* pcView,
}

//Xを進める
sPos.ForwardDrawCol(tabDispWidthLayout);
sPos.ForwardDrawCol(nTabLayoutWidth);
}

/*
Expand Down