Consolidated Statements of Comprehensive (Loss) Income (Parenthetical) - USD ($) $ in Thousands	12 Months Ended
Dec. 31, 2018	Dec. 31, 2017	Dec. 31, 2016
Statement of Comprehensive Income [Abstract]
Unrealized gain (loss) on marketable securities, tax	$ (18)	$ 28	$ 16

Amount of tax expense (benefit) before reclassification adjustments of unrealized holding gain (loss) on available-for-sale securities.

Reference 1: http://fasb.org/us-gaap/role/ref/legacyRef
-Publisher FASB
-Name Accounting Standards Codification
-Topic 220
-SubTopic 10
-Section 45
-Paragraph 12
-URI http://asc.fasb.org/extlink&oid=114867106&loc=d3e640-108580

Reference 2: http://www.xbrl.org/2003/role/disclosureRef
-Publisher FASB
-Name Accounting Standards Codification
-Topic 220
-SubTopic 10
-Section 45
-Paragraph 10A
-Subparagraph (f)
-URI http://asc.fasb.org/extlink&oid=114867106&loc=SL7669646-108580

Name:	us-gaap_OtherComprehensiveIncomeUnrealizedHoldingGainLossOnSecuritiesArisingDuringPeriodTax
Namespace Prefix:	us-gaap_
Data Type:	xbrli:monetaryItemType
Balance Type:	debit
Period Type:	duration