feat: Add Markdown File Support for Audiobook Generation

This commit is contained in:
Xia Dong
2025-09-02 15:14:13 +08:00
parent 023884d6d8
commit 67dbb0a8fb
2 changed files with 438 additions and 154 deletions
+278 -19
View File
@@ -43,18 +43,22 @@ class HandlerDialog(QDialog):
super().__init__(parent) super().__init__(parent)
# Determine file type if not explicitly provided # Determine file type if not explicitly provided
self.file_type = file_type or ( if file_type:
"pdf" if book_path.lower().endswith(".pdf") else "epub" self.file_type = file_type
) elif book_path.lower().endswith(".pdf"):
self.file_type = "pdf"
elif book_path.lower().endswith((".md", ".markdown")):
self.file_type = "markdown"
else:
self.file_type = "epub"
self.book_path = book_path self.book_path = book_path
# Extract book name from file path # Extract book name from file path
book_name = os.path.splitext(os.path.basename(book_path))[0] book_name = os.path.splitext(os.path.basename(book_path))[0]
# Set window title based on file type and book name # Set window title based on file type and book name
self.setWindowTitle( item_type = "Chapters" if self.file_type in ["epub", "markdown"] else "Pages"
f'Select {"Chapters" if self.file_type == "epub" else "Pages"} - {book_name}' self.setWindowTitle(f'Select {item_type} - {book_name}')
)
self.resize(1200, 900) self.resize(1200, 900)
self._block_signals = False # Flag to prevent recursive signals self._block_signals = False # Flag to prevent recursive signals
# Configure window: remove help button and allow resizing # Configure window: remove help button and allow resizing
@@ -69,7 +73,12 @@ class HandlerDialog(QDialog):
# Load the book based on file type # Load the book based on file type
try: try:
self.book = epub.read_epub(book_path) if self.file_type == "epub" else None if self.file_type == "epub":
self.book = epub.read_epub(book_path)
elif self.file_type == "markdown":
self.book = None # Markdown doesn't use ebooklib
else:
self.book = None
except KeyError as e: except KeyError as e:
logging.error( logging.error(
f"EPUB file is missing a referenced file: {e}. Skipping missing file." f"EPUB file is missing a referenced file: {e}. Skipping missing file."
@@ -100,6 +109,15 @@ class HandlerDialog(QDialog):
logging.error(f"Failed to patch ebooklib for missing files: {patch_e}") logging.error(f"Failed to patch ebooklib for missing files: {patch_e}")
raise e raise e
self.pdf_doc = fitz.open(book_path) if self.file_type == "pdf" else None self.pdf_doc = fitz.open(book_path) if self.file_type == "pdf" else None
self.markdown_text = None
if self.file_type == "markdown":
try:
encoding = self._detect_encoding(book_path)
with open(book_path, "r", encoding=encoding, errors="replace") as f:
self.markdown_text = f.read()
except Exception as e:
logging.error(f"Error reading markdown file: {e}")
self.markdown_text = ""
# Extract book metadata # Extract book metadata
self.book_metadata = self._extract_book_metadata() self.book_metadata = self._extract_book_metadata()
@@ -166,6 +184,17 @@ class HandlerDialog(QDialog):
# Update checkbox states # Update checkbox states
self._update_checkbox_states() self._update_checkbox_states()
def _detect_encoding(self, file_path):
"""Detect encoding of a text file"""
with open(file_path, "rb") as f:
raw_data = f.read()
try:
import chardet
result = chardet.detect(raw_data)
return result.get("encoding", "utf-8")
except ImportError:
return "utf-8"
def _preprocess_content(self): def _preprocess_content(self):
"""Pre-process content from the document""" """Pre-process content from the document"""
if self.file_type == "epub": if self.file_type == "epub":
@@ -178,6 +207,8 @@ class HandlerDialog(QDialog):
) )
# Fallback to a simpler spine-based processing if nav fails # Fallback to a simpler spine-based processing if nav fails
self._process_epub_content_spine_fallback() self._process_epub_content_spine_fallback()
elif self.file_type == "markdown":
self._preprocess_markdown_content()
else: else:
self._preprocess_pdf_content() self._preprocess_pdf_content()
@@ -199,10 +230,70 @@ class HandlerDialog(QDialog):
# (common in headers/footers like "- 42 -") # (common in headers/footers like "- 42 -")
text = re.sub(r"\s+[-–—]\s*\d+\s*[-–—]?\s*$", "", text, flags=re.MULTILINE) text = re.sub(r"\s+[-–—]\s*\d+\s*[-–—]?\s*$", "", text, flags=re.MULTILINE)
page_id = f"page_{page_num+1}" page_id = f"page_{page_num + 1}"
self.content_texts[page_id] = text self.content_texts[page_id] = text
self.content_lengths[page_id] = calculate_text_length(text) self.content_lengths[page_id] = calculate_text_length(text)
def _preprocess_markdown_content(self):
"""Pre-process markdown content and extract chapters/sections"""
if not self.markdown_text:
return
# Split markdown by headers to create chapters
import re
# Clean the text first
text = clean_text(self.markdown_text)
# Find all markdown headers (# ## ### etc.)
header_pattern = r'^(#{1,6})\s+(.+)$'
headers = list(re.finditer(header_pattern, text, re.MULTILINE))
self.content_texts = {}
self.content_lengths = {}
if not headers:
# No headers found, treat entire content as single chapter
chapter_id = "markdown_content"
self.content_texts[chapter_id] = text
self.content_lengths[chapter_id] = calculate_text_length(text)
return
# Process headers and extract content between them
for i, header_match in enumerate(headers):
header_level = len(header_match.group(1)) # Number of # symbols
header_text = header_match.group(2).strip()
header_start = header_match.start()
# Find content between this header and the next one of same or higher level
content_start = header_match.end()
content_end = len(text)
# Look for next header of same or higher level (fewer # symbols)
for j in range(i + 1, len(headers)):
next_header = headers[j]
next_level = len(next_header.group(1))
if next_level <= header_level:
content_end = next_header.start()
break
# Extract content for this chapter
chapter_content = text[content_start:content_end].strip()
# Create unique identifier for this chapter
chapter_id = f"markdown_chapter_{i + 1}"
# Store the chapter content
if chapter_content:
# Include the header in the content
full_content = f"{header_text}\n\n{chapter_content}"
self.content_texts[chapter_id] = full_content
self.content_lengths[chapter_id] = calculate_text_length(full_content)
else:
# Even if no content, store the header
self.content_texts[chapter_id] = header_text
self.content_lengths[chapter_id] = calculate_text_length(header_text)
def _process_epub_content_spine_fallback(self): def _process_epub_content_spine_fallback(self):
"""Fallback EPUB processing based purely on spine order.""" """Fallback EPUB processing based purely on spine order."""
logging.info("Using spine fallback for EPUB processing.") logging.info("Using spine fallback for EPUB processing.")
@@ -250,7 +341,7 @@ class HandlerDialog(QDialog):
title = h1.get_text(strip=True) title = h1.get_text(strip=True)
if not title: if not title:
title = f"Untitled Chapter {i+1}" title = f"Untitled Chapter {i + 1}"
synthetic_toc.append( synthetic_toc.append(
(epub.Link(doc_href, title, doc_href), []) (epub.Link(doc_href, title, doc_href), [])
) # Wrap in tuple and empty list for compatibility ) # Wrap in tuple and empty list for compatibility
@@ -847,6 +938,8 @@ class HandlerDialog(QDialog):
else: else:
logging.warning("Building EPUB tree using fallback book.toc.") logging.warning("Building EPUB tree using fallback book.toc.")
self._build_epub_tree_fallback(self.book.toc, self.treeWidget) self._build_epub_tree_fallback(self.book.toc, self.treeWidget)
elif self.file_type == "markdown":
self._build_markdown_tree()
else: else:
self._build_pdf_tree() self._build_pdf_tree()
@@ -993,7 +1086,7 @@ class HandlerDialog(QDialog):
if isinstance(page, int) if isinstance(page, int)
else self.pdf_doc.resolve_link(page)[0] else self.pdf_doc.resolve_link(page)[0]
) )
page_id = f"page_{page_num+1}" page_id = f"page_{page_num + 1}"
# attach chapters on same page under original # attach chapters on same page under original
if page_num in self.bookmark_items_map: if page_num in self.bookmark_items_map:
orig = self.bookmark_items_map[page_num] orig = self.bookmark_items_map[page_num]
@@ -1033,8 +1126,8 @@ class HandlerDialog(QDialog):
): ):
continue continue
page_id = f"page_{sub_page_num+1}" page_id = f"page_{sub_page_num + 1}"
page_title = f"Page {sub_page_num+1}" page_title = f"Page {sub_page_num + 1}"
page_text = self.content_texts.get(page_id, "").strip() page_text = self.content_texts.get(page_id, "").strip()
if page_text: if page_text:
@@ -1077,8 +1170,8 @@ class HandlerDialog(QDialog):
parent_item = ( parent_item = (
self.bookmark_items_map[prev_nums[-1]] if prev_nums else self.treeWidget self.bookmark_items_map[prev_nums[-1]] if prev_nums else self.treeWidget
) )
page_id = f"page_{page_num+1}" page_id = f"page_{page_num + 1}"
title = f"Page {page_num+1}" title = f"Page {page_num + 1}"
text = self.content_texts.get(page_id, "").strip() text = self.content_texts.get(page_id, "").strip()
if text: if text:
first = text.split("\n", 1)[0].strip() first = text.split("\n", 1)[0].strip()
@@ -1095,6 +1188,63 @@ class HandlerDialog(QDialog):
else: else:
page_item.setFlags(page_item.flags() & ~Qt.ItemIsUserCheckable) page_item.setFlags(page_item.flags() & ~Qt.ItemIsUserCheckable)
def _build_markdown_tree(self):
"""Build tree structure for markdown file based on headers"""
if not self.content_texts:
return
# If only one content item (no headers), create simple structure
if len(self.content_texts) == 1:
chapter_id = list(self.content_texts.keys())[0]
title = "Content"
item = QTreeWidgetItem(self.treeWidget, [title])
item.setData(0, Qt.UserRole, chapter_id)
if self.content_lengths.get(chapter_id, 0) > 0:
item.setFlags(item.flags() | Qt.ItemIsUserCheckable)
is_checked = chapter_id in self.checked_chapters
item.setCheckState(0, Qt.Checked if is_checked else Qt.Unchecked)
else:
item.setFlags(item.flags() & ~Qt.ItemIsUserCheckable)
return
# Build hierarchical tree based on markdown headers
# First, parse the original markdown to get header structure
import re
header_pattern = r'^(#{1,6})\s+(.+)$'
headers = list(re.finditer(header_pattern, self.markdown_text, re.MULTILINE))
# Create tree items for each header
stack = [] # Stack to track parent items at different levels
for i, header_match in enumerate(headers):
header_level = len(header_match.group(1))
header_text = header_match.group(2).strip()
chapter_id = f"markdown_chapter_{i + 1}"
# Pop stack until we find appropriate parent level
while stack and stack[-1][0] >= header_level:
stack.pop()
# Determine parent item
parent_item = stack[-1][1] if stack else self.treeWidget
# Create tree item
item = QTreeWidgetItem(parent_item, [header_text])
item.setData(0, Qt.UserRole, chapter_id)
# Set checkable state based on content
if self.content_lengths.get(chapter_id, 0) > 0:
item.setFlags(item.flags() | Qt.ItemIsUserCheckable)
is_checked = chapter_id in self.checked_chapters
item.setCheckState(0, Qt.Checked if is_checked else Qt.Unchecked)
else:
item.setFlags(item.flags() & ~Qt.ItemIsUserCheckable)
# Add to stack for potential children
stack.append((header_level, item))
def _build_pdf_pages_tree(self): def _build_pdf_pages_tree(self):
pages_item = QTreeWidgetItem(self.treeWidget, ["Pages"]) pages_item = QTreeWidgetItem(self.treeWidget, ["Pages"])
pages_item.setFlags(pages_item.flags() & ~Qt.ItemIsUserCheckable) pages_item.setFlags(pages_item.flags() & ~Qt.ItemIsUserCheckable)
@@ -1103,8 +1253,8 @@ class HandlerDialog(QDialog):
pages_item.setFont(0, font) pages_item.setFont(0, font)
for page_num in range(len(self.pdf_doc)): for page_num in range(len(self.pdf_doc)):
page_id = f"page_{page_num+1}" page_id = f"page_{page_num + 1}"
page_title = f"Page {page_num+1}" page_title = f"Page {page_num + 1}"
page_text = self.content_texts.get(page_id, "").strip() page_text = self.content_texts.get(page_id, "").strip()
if page_text: if page_text:
@@ -1166,7 +1316,7 @@ class HandlerDialog(QDialog):
buttons.accepted.connect(self.accept) buttons.accepted.connect(self.accept)
buttons.rejected.connect(self.reject) buttons.rejected.connect(self.reject)
item_type = "chapters" if self.file_type == "epub" else "pages" item_type = "chapters" if self.file_type in ["epub", "markdown"] else "pages"
self.auto_select_btn = QPushButton(f"Auto-select {item_type}", self) self.auto_select_btn = QPushButton(f"Auto-select {item_type}", self)
self.auto_select_btn.clicked.connect(self.auto_select_chapters) self.auto_select_btn.clicked.connect(self.auto_select_chapters)
@@ -1214,7 +1364,7 @@ class HandlerDialog(QDialog):
checkbox_text = ( checkbox_text = (
"Save each chapter separately" "Save each chapter separately"
if self.file_type == "epub" if self.file_type in ["epub", "markdown"]
else "Save each page separately" else "Save each page separately"
) )
self.save_chapters_checkbox = QCheckBox(checkbox_text, self) self.save_chapters_checkbox = QCheckBox(checkbox_text, self)
@@ -1275,7 +1425,7 @@ class HandlerDialog(QDialog):
checked_count = 0 checked_count = 0
if self.file_type == "epub": if self.file_type in ["epub", "markdown"]:
iterator = QTreeWidgetItemIterator(self.treeWidget) iterator = QTreeWidgetItemIterator(self.treeWidget)
while iterator.value(): while iterator.value():
item = iterator.value() item = iterator.value()
@@ -1367,6 +1517,8 @@ class HandlerDialog(QDialog):
if self.file_type == "epub": if self.file_type == "epub":
self._run_epub_auto_check() self._run_epub_auto_check()
elif self.file_type == "markdown":
self._run_markdown_auto_check()
else: else:
self._run_pdf_auto_check() self._run_pdf_auto_check()
@@ -1404,6 +1556,40 @@ class HandlerDialog(QDialog):
iterator += 1 iterator += 1
def _run_markdown_auto_check(self):
"""Auto-select markdown chapters with significant content"""
iterator = QTreeWidgetItemIterator(self.treeWidget)
while iterator.value():
item = iterator.value()
if not (item.flags() & Qt.ItemIsUserCheckable):
iterator += 1
continue
identifier = item.data(0, Qt.UserRole)
# Select chapters with content > 500 characters or parent items
has_significant_content = identifier and self.content_lengths.get(identifier, 0) > 500
is_parent = item.childCount() > 0
if has_significant_content or is_parent:
item.setCheckState(0, Qt.Checked)
# Also check children if this is a parent
if is_parent:
for i in range(item.childCount()):
child = item.child(i)
if child.flags() & Qt.ItemIsUserCheckable:
child_identifier = child.data(0, Qt.UserRole)
child_has_content = (
child_identifier and self.content_lengths.get(child_identifier, 0) > 0
)
child_is_parent = child.childCount() > 0
if child_has_content or child_is_parent:
child.setCheckState(0, Qt.Checked)
else:
item.setCheckState(0, Qt.Unchecked)
iterator += 1
def _run_pdf_auto_check(self): def _run_pdf_auto_check(self):
if hasattr(self, "has_pdf_bookmarks") and not self.has_pdf_bookmarks: if hasattr(self, "has_pdf_bookmarks") and not self.has_pdf_bookmarks:
iterator = QTreeWidgetItemIterator(self.treeWidget) iterator = QTreeWidgetItemIterator(self.treeWidget)
@@ -1626,6 +1812,41 @@ class HandlerDialog(QDialog):
if "cover" in item.get_name().lower(): if "cover" in item.get_name().lower():
metadata["cover_image"] = item.get_content() metadata["cover_image"] = item.get_content()
break break
elif self.file_type == "markdown":
# Extract metadata from markdown frontmatter or first heading
if self.markdown_text:
# Try to extract YAML frontmatter
frontmatter_match = re.match(r'^---\s*\n(.*?)\n---\s*\n', self.markdown_text, re.DOTALL)
if frontmatter_match:
try:
frontmatter = frontmatter_match.group(1)
# Simple YAML-like parsing for common fields
title_match = re.search(r'^title:\s*(.+)$', frontmatter, re.MULTILINE | re.IGNORECASE)
if title_match:
metadata["title"] = title_match.group(1).strip().strip('"\'')
author_match = re.search(r'^author:\s*(.+)$', frontmatter, re.MULTILINE | re.IGNORECASE)
if author_match:
metadata["authors"] = [author_match.group(1).strip().strip('"\'')]
desc_match = re.search(r'^description:\s*(.+)$', frontmatter, re.MULTILINE | re.IGNORECASE)
if desc_match:
metadata["description"] = desc_match.group(1).strip().strip('"\'')
date_match = re.search(r'^date:\s*(.+)$', frontmatter, re.MULTILINE | re.IGNORECASE)
if date_match:
date_str = date_match.group(1).strip().strip('"\'')
year_match = re.search(r'\b(19|20)\d{2}\b', date_str)
if year_match:
metadata["publication_year"] = year_match.group(0)
except Exception as e:
logging.warning(f"Error parsing markdown frontmatter: {e}")
# Fallback: use first H1 header as title if no frontmatter title
if not metadata["title"]:
first_h1_match = re.search(r'^#\s+(.+)$', self.markdown_text, re.MULTILINE)
if first_h1_match:
metadata["title"] = first_h1_match.group(1).strip()
else: else:
pdf_info = self.pdf_doc.metadata pdf_info = self.pdf_doc.metadata
if pdf_info: if pdf_info:
@@ -1670,6 +1891,8 @@ class HandlerDialog(QDialog):
def get_selected_text(self): def get_selected_text(self):
if self.file_type == "epub": if self.file_type == "epub":
return self._get_epub_selected_text() return self._get_epub_selected_text()
elif self.file_type == "markdown":
return self._get_markdown_selected_text()
else: else:
return self._get_pdf_selected_text() return self._get_pdf_selected_text()
@@ -1709,6 +1932,42 @@ class HandlerDialog(QDialog):
return "\n".join(metadata_tags) return "\n".join(metadata_tags)
def _get_markdown_selected_text(self):
"""Get selected text from markdown chapters"""
all_checked_identifiers = set()
chapter_texts = []
# Add metadata tags at the beginning
metadata_tags = self._format_metadata_tags()
item_order_counter = 0
ordered_checked_items = []
iterator = QTreeWidgetItemIterator(self.treeWidget)
while iterator.value():
item = iterator.value()
item_order_counter += 1
if item.checkState(0) == Qt.Checked:
identifier = item.data(0, Qt.UserRole)
if identifier and identifier != "info:bookinfo":
all_checked_identifiers.add(identifier)
ordered_checked_items.append((item_order_counter, item, identifier))
iterator += 1
ordered_checked_items.sort(key=lambda x: x[0])
for order, item, identifier in ordered_checked_items:
text = self.content_texts.get(identifier)
if text and text.strip():
title = item.text(0)
# Remove leading dashes from title
title = re.sub(r"^\s*[-–—]\s*", "", title).strip()
marker = f"<<CHAPTER_MARKER:{title}>>"
chapter_texts.append(marker + "\n" + text)
full_text = metadata_tags + "\n\n" + "\n\n".join(chapter_texts)
return full_text, all_checked_identifiers
def _get_epub_selected_text(self): def _get_epub_selected_text(self):
all_checked_identifiers = set() all_checked_identifiers = set()
chapter_texts = [] chapter_texts = []
+48 -23
View File
@@ -158,7 +158,7 @@ class InputBox(QLabel):
self.setAlignment(Qt.AlignCenter) self.setAlignment(Qt.AlignCenter)
self.setAcceptDrops(True) self.setAcceptDrops(True)
self.setText( self.setText(
"Drag and drop your file here or click to browse.\n(.txt, .epub, .pdf)" "Drag and drop your file here or click to browse.\n(.txt, .epub, .pdf, .md)"
) )
self.setStyleSheet( self.setStyleSheet(
f"QLabel {{ {self.STYLE_DEFAULT} }} QLabel:hover {{ {self.STYLE_DEFAULT_HOVER} }}" f"QLabel {{ {self.STYLE_DEFAULT} }} QLabel:hover {{ {self.STYLE_DEFAULT_HOVER} }}"
@@ -251,9 +251,11 @@ class InputBox(QLabel):
return str(n) return str(n)
if ( if (
file_path.lower().endswith(".epub") or file_path.lower().endswith(".pdf") file_path.lower().endswith(".epub")
or file_path.lower().endswith(".pdf")
or file_path.lower().endswith((".md", ".markdown"))
) and hasattr(self.window(), "selected_chapters"): ) and hasattr(self.window(), "selected_chapters"):
# EPUB or PDF: sum character counts for selected chapters # EPUB, PDF, or Markdown: sum character counts for selected chapters
try: try:
book_path = file_path book_path = file_path
@@ -345,7 +347,7 @@ class InputBox(QLabel):
None # Reset the displayed file path when clearing input None # Reset the displayed file path when clearing input
) )
self.setText( self.setText(
"Drag and drop your file here or click to browse.\n(.txt, .epub, .pdf)" "Drag and drop your file here or click to browse.\n(.txt, .epub, .pdf, .md)"
) )
self.setStyleSheet( self.setStyleSheet(
f"QLabel {{ {self.STYLE_DEFAULT} }} QLabel:hover {{ {self.STYLE_DEFAULT_HOVER} }}" f"QLabel {{ {self.STYLE_DEFAULT} }} QLabel:hover {{ {self.STYLE_DEFAULT_HOVER} }}"
@@ -384,6 +386,7 @@ class InputBox(QLabel):
ext.endswith(".txt") ext.endswith(".txt")
or ext.endswith(".epub") or ext.endswith(".epub")
or ext.endswith(".pdf") or ext.endswith(".pdf")
or ext.endswith((".md", ".markdown"))
): ):
event.acceptProposedAction() event.acceptProposedAction()
# Set hover style based on current state # Set hover style based on current state
@@ -433,20 +436,26 @@ class InputBox(QLabel):
) )
self.set_file_info(file_path) self.set_file_info(file_path)
event.acceptProposedAction() event.acceptProposedAction()
elif file_path.lower().endswith(".epub") or file_path.lower().endswith( elif (file_path.lower().endswith(".epub")
".pdf" or file_path.lower().endswith(".pdf")
): or file_path.lower().endswith((".md", ".markdown"))):
# Determine file type
if file_path.lower().endswith(".epub"):
file_type = "epub"
elif file_path.lower().endswith(".pdf"):
file_type = "pdf"
else:
file_type = "markdown"
# Just store the file path but don't set the file info yet # Just store the file path but don't set the file info yet
win.selected_file_type = ( win.selected_file_type = file_type
"epub" if file_path.lower().endswith(".epub") else "pdf"
)
win.selected_book_path = file_path win.selected_book_path = file_path
win.open_book_file( win.open_book_file(
file_path # This will handle the dialog and setting file info file_path # This will handle the dialog and setting file info
) )
event.acceptProposedAction() event.acceptProposedAction()
else: else:
self.set_error("Please drop a .txt, .epub, or .pdf file.") self.set_error("Please drop a .txt, .epub, .pdf, or .md file.")
event.ignore() event.ignore()
else: else:
event.ignore() event.ignore()
@@ -1155,16 +1164,21 @@ class abogen(QWidget):
return return
try: try:
file_path, _ = QFileDialog.getOpenFileName( file_path, _ = QFileDialog.getOpenFileName(
self, "Select File", "", "Supported Files (*.txt *.epub *.pdf)" self, "Select File", "", "Supported Files (*.txt *.epub *.pdf *.md)"
) )
if not file_path: if not file_path:
return return
if file_path.lower().endswith(".epub") or file_path.lower().endswith( if (file_path.lower().endswith(".epub")
".pdf" or file_path.lower().endswith(".pdf")
): or file_path.lower().endswith((".md", ".markdown"))):
self.selected_file_type = ( # Determine file type
"epub" if file_path.lower().endswith(".epub") else "pdf" if file_path.lower().endswith(".epub"):
) self.selected_file_type = "epub"
elif file_path.lower().endswith(".pdf"):
self.selected_file_type = "pdf"
else:
self.selected_file_type = "markdown"
self.selected_book_path = file_path self.selected_book_path = file_path
# Don't set file info immediately, open_book_file will handle it after dialog is accepted # Don't set file info immediately, open_book_file will handle it after dialog is accepted
if not self.open_book_file(file_path): if not self.open_book_file(file_path):
@@ -1190,7 +1204,10 @@ class abogen(QWidget):
self.last_opened_book_path = book_path self.last_opened_book_path = book_path
dialog = HandlerDialog( dialog = HandlerDialog(
book_path, checked_chapters=self.selected_chapters, parent=self book_path,
file_type=getattr(self, 'selected_file_type', None),
checked_chapters=self.selected_chapters,
parent=self
) )
dialog.setWindowModality(Qt.NonModal) dialog.setWindowModality(Qt.NonModal)
dialog.setModal(False) dialog.setModal(False)
@@ -1201,10 +1218,18 @@ class abogen(QWidget):
return False return False
chapters_text, all_checked_hrefs = dialog.get_selected_text() chapters_text, all_checked_hrefs = dialog.get_selected_text()
if not all_checked_hrefs: if not all_checked_hrefs:
file_type = "pdf" if book_path.lower().endswith(".pdf") else "epub" # Determine file type for error message
error_msg = ( if book_path.lower().endswith(".pdf"):
f"No {'pages' if file_type == 'pdf' else 'chapters'} selected." file_type = "pdf"
) item_type = "pages"
elif book_path.lower().endswith((".md", ".markdown")):
file_type = "markdown"
item_type = "chapters"
else:
file_type = "epub"
item_type = "chapters"
error_msg = f"No {item_type} selected."
self._show_error_message_box(f"{file_type.upper()} Error", error_msg) self._show_error_message_box(f"{file_type.upper()} Error", error_msg)
return False return False
self.selected_chapters = all_checked_hrefs self.selected_chapters = all_checked_hrefs