Skip to content

Update of About window + format#24

Closed
Sylvain78 wants to merge 0 commit into
HaikuArchives:masterfrom
Sylvain78:master
Closed

Update of About window + format#24
Sylvain78 wants to merge 0 commit into
HaikuArchives:masterfrom
Sylvain78:master

Conversation

@Sylvain78
Copy link
Copy Markdown
Contributor

-Improvements in AboutWindow : get rid of bebits page, update url for donate/homepage.
-Code formatting

Comment thread Src/Headers/Constants.h Outdated
K_PREFS_WINDOW_QUIT = 'pfwq'
K_PREFS_VIEW_RESET_COLOR_DEFAULTS = 'pcrd',
K_PREFS_VIEW_RESET_COMMAND_DEFAULTS = 'pcmd',
K_PREFS_VIEW_RESET_TOOLBAR_DEFAULTS = 'pvrt',
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Traling spaces/tabs.

Comment thread Src/Source/AboutWindow.cpp Outdated
m_beBits = new BButton(
btnRect, "fBeBitsbtn", "BeBits Page", new BMessage(AboutMessages::K_GOTO_BETEX_BEBITSPAGE));
m_gradient->AddChild(m_beBits);
/* Long gone ....
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Just removing the dead code might be better.

Comment thread Src/Source/MainWindow.cpp
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

The PR description doesn't mentions why these changes in this file are necessary. The messy commit history makes it hard to see if it was mentioned in a commit message.

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

I think this change should be reerted.

@korli
Copy link
Copy Markdown
Contributor

korli commented Dec 21, 2025

Please use a rebased branch for new commits instead of the master branch.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

3 participants