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

Title bar for Windows & Linux #759

Merged
merged 19 commits into from
Mar 4, 2018
Merged

Title bar for Windows & Linux #759

Show file tree
Hide file tree
Changes from 1 commit
Commits
Show all changes
19 commits
Select commit Hold shift + click to select a range
cdf4776
[WIP] incorporate electron-react-titlebar
haraldox Feb 20, 2018
fc76e14
choose platform with environment variable `FRANZ_PLATFORM`
haraldox Feb 20, 2018
08536e7
fix CSS conflicts
haraldox Feb 20, 2018
fd36235
fix observable template
haraldox Feb 21, 2018
0f1b958
[MINOR] rename `template` to `currentTemplate`
haraldox Feb 21, 2018
5157ce0
remove electron frame for Windows testing on macOS
haraldox Feb 21, 2018
136be27
override platform in code and simulate different environment
haraldox Feb 22, 2018
af38a63
Merge branch 'fix/copy-paste' into feature/titlebar
haraldox Feb 22, 2018
ae1014e
complete titlebar for Windows & Linux
haraldox Feb 23, 2018
afa43c1
[MINOR] switch to win32 testing on all platforms
haraldox Feb 23, 2018
e2d995d
fix .css import
adlk Feb 23, 2018
1b68fda
Merge branch 'feature/titlebar' of github.com:meetfranz/franz into fe…
adlk Feb 23, 2018
aa655ac
ADD Franz-wide internationalization
haraldox Mar 1, 2018
41ce9d8
fix linting errors
haraldox Mar 1, 2018
1d2df4a
internationalize validation messages
haraldox Mar 2, 2018
662b55b
Merge branch 'feature/titlebar' into feature/i18n-universal
haraldox Mar 2, 2018
a49511f
internationalized menu
haraldox Mar 2, 2018
dcc2dd3
REMOVE autoHideMenuBar for Windows
haraldox Mar 2, 2018
a17cfb3
Finalize titlebar styling
adlk Mar 3, 2018
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
Prev Previous commit
Next Next commit
fix observable template
  • Loading branch information
@haraldox
haraldox committed Feb 21, 2018
commit fd36235fe44dd9c57669e1f529b9bb56774ece6b
1 change: 0 additions & 1 deletion src/components/layout/AppLayout.js
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -6,7 +6,6 @@ import { TitleBar } from 'electron-react-titlebar';

import InfoBar from '../ui/InfoBar';
import globalMessages from '../../i18n/globalMessages';
import Menu from '../../lib/Menu';

function createMarkup(HTMLString) {
return { __html: HTMLString };
Expand Down
8 changes: 5 additions & 3 deletions src/lib/Menu.js
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -116,6 +116,7 @@ const template = [

export default class FranzMenu {
@observable tpl = template;
@observable template = null;

constructor(stores, actions) {
this.stores = stores;
Expand All @@ -124,12 +125,12 @@ export default class FranzMenu {
autorun(this._build.bind(this));
}

@computed get template() {
return toJS(this.tpl);
get template() {
return this.template;
}

_build() {
const tpl = this.template;
const tpl = toJS(this.tpl);

tpl[1].submenu.push({
role: 'toggledevtools',
Expand Down Expand Up @@ -252,6 +253,7 @@ export default class FranzMenu {
tpl[isMac ? 3 : 2].submenu = toJS(this.serviceTpl);
}

this.template = tpl;
const menu = Menu.buildFromTemplate(tpl);
Menu.setApplicationMenu(menu);
}
Expand Down