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

Introduce React, TypeScript, TSLint and React-StyleGuidist #2219

Merged
merged 23 commits into from
Apr 6, 2018
Merged

Introduce React, TypeScript, TSLint and React-StyleGuidist #2219

Show file tree
Hide file tree
Changes from 1 commit
Commits
Show all changes
23 commits
Select commit Hold shift + click to select a range
26f69b6
Gruntfile: Remove edits to background.js; no longer applicable
scottnonnenberg-signal Mar 22, 2018
e32dad2
Gruntfile: limit jshint/jscs watch tasks to own files, not *.js
scottnonnenberg-signal Mar 22, 2018
1ad2b08
Move moment from bower deps to npm deps
scottnonnenberg-signal Mar 24, 2018
f86a6ef
Update style of i18n, pull it in via preload instead of .html
scottnonnenberg-signal Mar 24, 2018
893fb1c
Introduce React/TypeScript/TSLint into app for new UI components
scottnonnenberg-signal Mar 26, 2018
1326b26
Full styleguide now available via `yarn styleguide`
scottnonnenberg-signal Apr 3, 2018
3899eb8
Add 'use strict' to i18n.js
scottnonnenberg-signal Apr 4, 2018
104a1ca
Remove i18n.js from jshint run, fix missing semicolon
scottnonnenberg-signal Apr 5, 2018
ed12e1b
Add mp3 to project, make binary files available in rsg on util
scottnonnenberg-signal Apr 5, 2018
887abd7
Add missing shim, fix typo
scottnonnenberg-signal Apr 5, 2018
be91a89
Parse theme from querystring, put on util. Moment from require()
scottnonnenberg-signal Apr 5, 2018
1052341
Fix tslint errors and add tslint to CI task in package.json
scottnonnenberg-signal Apr 5, 2018
23293a3
New tslint rule: Interfaces must not start with I
scottnonnenberg-signal Apr 5, 2018
bb73d47
Gruntfile: Sort lines in jshint/jscs config
scottnonnenberg-signal Apr 5, 2018
2384092
Use Array.isArray in i18n.js
scottnonnenberg-signal Apr 5, 2018
61a7846
Fix typo in index.ts
scottnonnenberg-signal Apr 5, 2018
cdd16c9
Rename test-only MessageParents to ConversationContext
scottnonnenberg-signal Apr 5, 2018
e5322e8
Finish rename of MessageParents, re-add moment to page
scottnonnenberg-signal Apr 5, 2018
76be135
js/i18n.js -> js/modules/i18n.js
scottnonnenberg-signal Apr 5, 2018
50d4dba
Remove 'use strict' from i18n.js
scottnonnenberg-signal Apr 5, 2018
2353754
Big refactor: ts/ directory for all typescript, including react
scottnonnenberg-signal Apr 5, 2018
0530323
window.Signal.React -> window.Signal.Components
scottnonnenberg-signal Apr 5, 2018
96bd90a
Simplify assignment; add warning to preload.js about Style Guide
scottnonnenberg-signal Apr 6, 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
1 change: 0 additions & 1 deletion .eslintignore
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -16,7 +16,6 @@ test/views/*.js
# typescript-generated files
ts/**/*.js


# ES2015+ files
!js/background.js
!js/backup.js
Expand Down
2 changes: 1 addition & 1 deletion preload.js
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -150,7 +150,7 @@ window.Signal.Database = require('./js/modules/database');
window.Signal.Debug = require('./js/modules/debug');
window.Signal.Logs = require('./js/modules/logs');

window.Signal.React = {};
window.Signal.Components = {};

window.Signal.Migrations = {};
window.Signal.Migrations.deleteAttachmentData =
Expand Down
2 changes: 1 addition & 1 deletion test/styleguide/legacy_bridge.js
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -18,7 +18,7 @@ window.Signal.Migrations = {
getPlaceholderMigrations: () => {},
};

window.Signal.React = {};
window.Signal.Components = {};

window.EmojiConvertor = function EmojiConvertor() {};
window.EmojiConvertor.prototype.init_colons = () => {};
Expand Down
6 changes: 3 additions & 3 deletions ts/test/StyleGuideUtil.ts
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -77,7 +77,7 @@ parent.moment.locale(locale);
parent.React = React;
parent.ReactDOM = ReactDOM;

const SignalReact = parent.Signal.React = parent.Signal.React || {};
const SignalComponents: any = parent.Signal.Components = {};

SignalReact.Message = Message;
SignalReact.Reply = Reply;
SignalComponents.Message = Message;
SignalComponents.Reply = Reply;
Copy link
Contributor

Choose a reason for hiding this comment

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

Minor: What if we did the following?

parent.Signal.Components = {
  Message,
  Reply,
};